Skip to content

задать ширину drawer по размерам - #138

Open
khaliulin wants to merge 3 commits into
mainfrom
size.drawer.update
Open

задать ширину drawer по размерам#138
khaliulin wants to merge 3 commits into
mainfrom
size.drawer.update

Conversation

@khaliulin

Copy link
Copy Markdown
Contributor

Ширины боковой панели заданы по таблице Figma (коллекция sizing, переменная width): sm 280, base 350, lg 420, xlg 630.

До этого правил для p-drawer-sm/lg/xlg в CSS кита не было вовсе — компонент навешивал класс размера, но обрабатывать его было некому, и все размеры давали одну ширину 400px.

Что изменено

src/lib/providers/prime-preset/tokens/components/drawer.ts:

Класс Токен px
.p-drawer-left/right sizing.175x 350
…&.p-drawer-sm sizing.140x 280
…&.p-drawer-lg sizing.210x 420
…&.p-drawer-xlg sizing.315x 630

Проверено в браузере на story Components/Overlay/Drawer → Sizes, каждый размер в чистой загрузке: 350 / 280 / 420 / 630 px.

Гейты: check-component.mjs drawer — 0 FAIL, 2 WARN (advisory G8: extend.width, extend.scale не применены); check-dt-refs.mjs — мёртвых ссылок drawer нет; prettier и tsc --noEmit чисто.

Вопросы к дизайну

  1. base: 350 против 400 в экспорте. В tokens.json у drawer единственная ширина — extend.width = {dimension.size.2000} = 25rem = 400px, а таблица sizing.width даёт base = 350. Здесь взято значение из таблицы, поэтому extend.width остался неиспользованным (первый WARN).
  2. Нет ветки размеров в экспорте. Значения взяты со ступеней примитивов (sizing.140x/175x/210x/315x), как это уже сделано в dialog.ts. По-хорошему они должны приезжать реэкспортом как drawer.extend.overlayWidth.*.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant