[Feature][Gallery]: Возможность изменения кликабельной зоны стрелок
Opened this issue · 10 comments
Описание
Скриншоты
No response
Добавляем проп, который позволяет сократить высоту кликабельной зоны стрелок
Назвал Arrows: Fill | Fit. Предложения по неймингу принимаются.
Добавляем проп, который позволяет сократить высоту кликабельной зоны стрелок Назвал Arrows: Fill | Fit. Предложения по неймингу принимаются.
@qurle в дизайне получится назвать Arrow Clicable Area? По задаче Arrows не соответствует семантике
+ это к разговору о сохрании консистентности названий между кодом и дизайном
@inomdzhon
Я за семантическое название, но это довольно длинное. Если будут более лаконичные варики, я бы закоммитился.
@qurle Могу предложить варианты arrowArea, arrowHitbox, arrowZone. Какой нравится больше?
@EldarMuhamethanov, крутяк
Я бы почекал, что более принято: Zone или Area, и остановился на этом варианте
Давайте значения тоже изменим, 'fill' и 'fit' не звучит как что-то про Zone/Area
@EldarMuhamethanov, крутяк Я бы почекал, что более принято: Zone или Area, и остановился на этом варианте
Я бы все-таки выбрал arrowArea, кажется более привычным
@qurle Есть идеи по наименованиями значений? У меня есть парочка 'fullHeight'/'padded', 'stretch'/'compact'
На мой взгляд, 'fullHeight'/'padded' длинновато и нестандартно (тоже думал о похожем варианте). 'stretch'/'compact' — не хочется смешивать термины из адпативности.
Fill / Fit или кажется нормальным, потому что нет другой трактовки + термины частично из фигмы (там есть Fill / Hug), iOS (там есть Fill Container / Hug Content) и цсс (там есть fill-available / fit-content).
В общем,fill, stretch, full / hug, fit кажется простым и понятным.
Если вопрос конкретности, то можно в реакте завести проп как arrowAreaHeight = fill / fit.
@qurle
В целом идея с arrowAreaHeight звучит неплохо. Тогда в дизайне делаем arrowArea, а в коде arrowAreaHeight с указание, что в дизайне это arrowArea.