size.drawer: ширина панели и подложка из токенов - #125
Open
khaliulin wants to merge 1 commit into
Open
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. Компонент был помечен в плане как «заблокированный экспортом» — четыре мёртвые ссылки на размерные токены. Разбор показал, что из-за них ширина панели не задавалась вовсе.Что сделано
Ширина панели восстановлена.
drawer.root.width,drawer.sm.width,drawer.lg.width,drawer.xlg.widthв экспорте отсутствуют — все четыреdt()резолвились в пустоту, правила с ними CSS отбрасывал целиком. То есть боковая панель до сих пор рисовалась дефолтной шириной PrimeNG, а не по дизайну. В экспорте естьdrawer.extend.width(25rem) — базовая ширина переведена на него, замер: 400px.Правила
sm/lg/xlgудалены. Токенов под них в экспорте нет, а мёртвые правила не работали и до этой ветки. Размерные варианты вернутся вместе с реэкспортом — см. «Вопросы к дизайну».Сырые значения заменены токенами:
border-bottom: 1px→drawer.extend.borderWidth(закрывает и WARN G8)dimension.space.200→drawer.extend.margin— у компонента есть собственный токен на это100vw/100vhв полноэкранном режиме →100%: панель позиционированаfixed, результат тот же, а гейт не считает это шкалой дизайнаПрименён
drawer.extend.backdrop— фон подложки, который нигде не читался. Замер:rgba(0, 0, 0, 0.4).Файл экспортировал функцию через
export { drawerCss }в конце — гейт G4 считал это отсутствием экспорта. Переведено на обычныйexport const.Убраны комментарии-разделители по правилу проекта.
Проверки
check-component drawer— было 8 FAIL (4×G1 мёртвые ссылки, 2×G2 сырые размеры, G21px, G4 экспорт) и 4 WARN, стало 0 FAIL, 1 WARNcheck-stories-bg drawer— подложек нетcheck-dt-refs— мёртвых ссылок вdrawer.tsбольше нет (были все четыре из общего списка по киту)rgba(0,0,0,0.4), консоль без ошибокprettier --checkчистtokens/components/drawer.tsВопросы к дизайну
drawer.sm/lg/xlg.width— только одна ширинаextend.width. Если варианты нужны (обёртка принимаетsize), их надо завести в Figma; сейчас все размеры визуально одинаковы.drawer.extend.scale(dimension.size.50) — единственный оставшийся WARN G8: к чему относится масштаб, из экспорта не выводится, применить некуда.