Skip to content

size.skeleton: минимальная ширина без слоя синонимов - #117

Merged
AxyIX merged 1 commit into
mainfrom
size.skeleton
Aug 25, 2026
Merged

size.skeleton: минимальная ширина без слоя синонимов#117
AxyIX merged 1 commit into
mainfrom
size.skeleton

Conversation

@khaliulin

Copy link
Copy Markdown
Contributor

Ветка от main. Последний компонент из группы «слой синонимов + сырые var()» — после avatar, badge, chip, listbox.

Что сделано

Снят блок :root — две переменные --p-skeleton-extend-* и чтение одной из них сырым var(). Минимальная ширина берётся через dt('skeleton.extend.minWidth'). Файл сократился до одного правила: всё остальное (фон, анимация, радиус) применяет сам PrimeNG из токенов пресета.

Stories. У четырёх примеров снята подложка bg-surface-ground.

Проверки

  • check-component skeleton — было 2 FAIL (G3 сырая var, G5 слой синонимов), стало 0 FAIL, 1 WARN G8
  • check-stories-bg skeleton — подложек нет
  • check-dt-refs — мёртвых ссылок в skeleton.ts нет
  • рантайм: min-width 32px из токена, фон #e2e2e4, мёртвых --p-skeleton-* нет, консоль без ошибок
  • prettier --check чист
  • границы ветки: tokens/components/skeleton.ts и stories/components/skeleton/examples/

Сверка с Figma <Skeleton> (5221:12465, пересчёт базы 14 → 16): размер 32×32px, фон #e2e2e4, фон анимации #f0f0f1 — совпадают.

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

  • skeleton.extend.height (dimension.size.900 = 2rem) не применён — это WARN G8. Высоту скелетона PrimeNG проставляет инлайновым стилем из @Input() height, а обёртка кита по умолчанию отдаёт 1rem; CSS-правилом это не перекрыть, не сломав явно заданную высоту. В макете высота 28px (2rem при базе 14), то есть дефолт обёртки расходится с дизайном вдвое. Правка — в обёртке (skeleton.component.ts), это отдельная ветка вне зоны токенов.
  • радиус. skeleton.root.borderRadius = dimension.borderRadius.400 (0.875rem) против 7px в макете (0.5rem при базе 14). Четвёртый компонент с расхождением по этой ступени после menu, avatar, listbox.

@AxyIX
AxyIX merged commit 755df96 into main Aug 25, 2026
2 checks passed
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.

3 participants