VKUI icon indicating copy to clipboard operation
VKUI copied to clipboard

[Breaking changes] Окончательно отказаться от `Appearance` в пользу `vkui-tokens`

Open eugpoloz opened this issue 3 years ago • 1 comments

  • [ ] переводим все компоненты на токены
  • [ ] отказываемся в CSS от оформления привязанное к платформенному классу в пользу токенов из vkui-tokens
    /* ❌  bad */
    .Component {
      border-radius: 8px;
    }
    .Component--ios {
      border-radius: 10px;
    }
    /* ✅  good */
    .Component {
      border-radius: var(--vkui--size_border_radius--regular);
    }         
    
  • [ ] удаляем лишние селекторы, остававшиеся для обратной совместимости
  • [ ] решаем вопрос с наследованием темы (см. Платформы и темы в документации => Наследование темы). Вероятно, потом стоит это упомянуть и в https://github.com/VKCOM/VKUI/issues/2351

eugpoloz avatar Apr 12 '22 15:04 eugpoloz

Принёс вам ещё одну проблему предлагаемого подхода, без исправления или понимания которой, на мой взгляд, раскатывать не стоит.

Сейчас CSS собирается вместе с фоллбеками, и получается, что старые браузеры будут использовать библиотечные значения. В итоге получаем неконсистентный (и, скорее всего, кривой) интерфейс между старыми и новыми браузерами.

worldwidebaby avatar May 16 '22 05:05 worldwidebaby