size.popover: стрелка из токенов вместо подогнанных значений - #118
Open
khaliulin wants to merge 1 commit into
Open
size.popover: стрелка из токенов вместо подогнанных значений#118khaliulin wants to merge 1 commit into
khaliulin wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ветка от
main. Первый компонент группы «значения, подогнанные глазом под базу 14» — он же первый пункт таблицы §6 плана.Что сделано
Стрелка построена из токенов. В файле стояли два подогнанных значения:
border-width: 0.73remиmargin-left: -0.86rem(10.22px и −12.04px при базе 14). При переходе на 16px они дают стрелку шириной ~23px против 10.5px в макете — вдвое шире.Aura строит стрелку из
popover.gutter, но у насgutter= 0.25rem (зазор до триггера), и по нему стрелка вышла бы 4px. Для стрелки в экспорте есть свои токены —popover.extend.arrow.width(0.75rem) иpopover.extend.arrow.height(0.5rem), ровно размеры узлаarrowв Figma (10.5 × 7 при базе 14). Оба до этой ветки не применялись — гейт G8 их и показывал.Теперь треугольник считается из них: горизонтальный
border-width= половина ширины, вертикальный = высота. Внутренний треугольник (::after, цвет фона) меньше внешнего (::before, цвет границы) наpopover.extend.borderWidth— третий неприменённый токен, который тоже закрыт.Stories. У примеров убран
appendTo="self": PrimeNG в этой версии на таком значении пытается присоединить контейнер к самому себе и валится сCannot append [object HTMLDivElement] to selfпри каждом открытии. С дефолтнымbodypopover открывается и позиционируется корректно, консоль чистая.Проверки
check-component popover— было 1 FAIL (G2 сырой размер) и 2 WARN G8, стало 0 FAIL, 0 WARNcheck-stories-bg popover— подложек нетcheck-dt-refs— мёртвых ссылок вpopover.tsнетprettier --checkчистtokens/components/popover.tsиstories/components/popover/examples/Сверка с Figma
<Popover>(533:137, пересчёт базы 14 → 16): стрелка 12×8px,paddingконтента 12px, радиус 8px, фон#ffffff, граница#cecfd2, тень0 4px 8px— совпадают.Вопросы к дизайну
tokens.jsonне правил (экспорт из Figma):popover.root.gutter=dimension.space.100(0.25rem = 4px) меньше высоты стрелки (8px), поэтому стрелка наезжает на триггер на 8px. У Aura на этом месте 10px. Либоgutterдолжен быть не меньшеarrow.height, либо PrimeNG должен учитывать высоту стрелки при позиционировании — вопрос к дизайну и к обёртке.popover.content.padding=dimension.space.400(0.875rem = 14px) против 10.5px в макете (0.75rem при базе 14). То же расхождение ступениspace.400, что нашлось вtooltip.