Skip to content

size.date-picker: мёртвые var Aura заменены токенами кита - #124

Open
khaliulin wants to merge 1 commit into
mainfrom
size.date-picker
Open

size.date-picker: мёртвые var Aura заменены токенами кита#124
khaliulin wants to merge 1 commit into
mainfrom
size.date-picker

Conversation

@khaliulin

Copy link
Copy Markdown
Contributor

Ветка от main. Группа «сырые var() без слоя синонимов».

Что сделано

Три сырые var(--p-*) заменены на токены. Все три ссылались на semantic-переменные Aura (form.field.*, icon.size), которых наш пресет не выпускает: кит читает собственный экспорт, а не набор Aura, поэтому переменные были пустыми, а правила — мёртвыми.

  • --p-form-field-padding-xinputtext.root.paddingX (позиция крестика очистки)
  • --p-icon-sizeinputtext.extend.iconSize (сдвиг крестика, когда есть иконка календаря)
  • --p-form-field-lg-font-sizeinputtext.root.lg.fontSize (размер иконок у поля lg)

Применены два токена, которые дизайн задал, а кит не использовал (WARN G8):

  • datepicker.extend.extDate.selectedHoverBackground — наведение на уже выбранную дату не имело своего фона
  • datepicker.extend.extTitle.width — минимальная ширина заголовка «месяц + год»; без неё шапка календаря прыгала по ширине при смене месяца. Замер: 336px

Убраны комментарии-разделители по правилу проекта.

Проверки

  • check-component date-picker — было 3 FAIL (G3) и 7 WARN, стало 0 FAIL, 8 WARN (все G6)
  • check-stories-bg date-picker — подложек нет
  • check-dt-refs — мёртвых ссылок нет
  • рантайм: панель открывается, заголовок 336px, день 32×32px, консоль без ошибок. Из необъявленных --p-* — только чужие: три из button.ts (PR size.button: перевод на новые токены и базу 16px #97) и --p-select-checkmark-content из select.ts (PR size.select: галочка и фон опции из токенов #123), обе ветки ещё не влиты
  • prettier --check чист
  • границы ветки: один файл tokens/components/date-picker.ts

Осознанные WARN

  • G6, ссылки на button.* и inputtext.* — календарь переиспользует кнопки навигации и поле ввода; связь намеренная и теперь явная: раньше те же значения брались из несуществующих переменных Aura.

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