Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Feature][Gallery]: Возможность изменения кликабельной зоны стрелок #7860

Open
MrZillaGold opened this issue Oct 28, 2024 · 7 comments · May be fixed by #7907
Open

[Feature][Gallery]: Возможность изменения кликабельной зоны стрелок #7860

MrZillaGold opened this issue Oct 28, 2024 · 7 comments · May be fixed by #7907
Assignees
@EldarMuhamethanov
Labels
cmp:gallery type:feature
Milestone
v7.1.0

Comments

@MrZillaGold
Copy link
Contributor

Описание

image

Скриншоты

No response
@BlackySoul BlackySoul added this to the v7.1.0 milestone Oct 29, 2024
@EldarMuhamethanov EldarMuhamethanov self-assigned this Nov 1, 2024
@qurle
Copy link
Contributor

qurle commented Nov 6, 2024

Добавляем проп, который позволяет сократить высоту кликабельной зоны стрелок
Назвал Arrows: Fill | Fit. Предложения по неймингу принимаются.

Макет в фигме ↗

@EldarMuhamethanov EldarMuhamethanov linked a pull request Nov 7, 2024 that will close this issue
Open
5 tasks
@inomdzhon
Copy link
Contributor

inomdzhon commented Nov 7, 2024 •
edited
Loading

Добавляем проп, который позволяет сократить высоту кликабельной зоны стрелок Назвал Arrows: Fill | Fit. Предложения по неймингу принимаются.

Макет в фигме ↗

@qurle в дизайне получится назвать Arrow Clicable Area? По задаче Arrows не соответствует семантике

+ это к разговору о сохрании консистентности названий между кодом и дизайном

@qurle
Copy link
Contributor

qurle commented Nov 7, 2024

@inomdzhon
Я за семантическое название, но это довольно длинное. Если будут более лаконичные варики, я бы закоммитился.

@EldarMuhamethanov
Copy link
Contributor

@qurle Могу предложить варианты arrowArea, arrowHitbox, arrowZone. Какой нравится больше?

@qurle
Copy link
Contributor

qurle commented Nov 7, 2024

@EldarMuhamethanov, крутяк
Я бы почекал, что более принято: Zone или Area, и остановился на этом варианте

@inomdzhon
Copy link
Contributor

Давайте значения тоже изменим, 'fill' и 'fit' не звучит как что-то про Zone/Area

@EldarMuhamethanov
Copy link
Contributor

@EldarMuhamethanov, крутяк Я бы почекал, что более принято: Zone или Area, и остановился на этом варианте

Я бы все-таки выбрал arrowArea, кажется более привычным

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EldarMuhamethanov EldarMuhamethanov

Labels
cmp:gallery type:feature
Projects
VKUI
Status: 👀 In Review
Milestone
v7.1.0
Development

Successfully merging a pull request may close this issue.

feat(Gallery): add prop arrowClickableArea to arrows VKCOM/VKUI
5 participants
@inomdzhon @BlackySoul @qurle @MrZillaGold @EldarMuhamethanov