Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BREAKING CHANGE][Epic]: синхронизировать названия параметров с дизайном #7660

Closed
25 tasks done
inomdzhon opened this issue Sep 25, 2024 · 4 comments

Comments

@inomdzhon
Copy link
Contributor

inomdzhon commented Sep 25, 2024

Описание

Необходимо проанализировать дизайн в Figma и найти расхождения в именование параметров.

Note

Задачу упростит вот 🔗 этот документ, который @BlackySoul собрала с помощью самописного скрипта.

Можно будет у неё уточнить по какому принципу сопоставлялся код и дизайн.

Логика разрешения расхождения в именовании

⚠️ IMPORTANT 1

По поводу под-компонента есть нюанс, что одно дело если это некий сервисный компонент, как в случае с компонентом Group это Group.Expanded или как с Snackbar это Snackbar.Basic, другое если под-компонент мог бы выступать как параметр компонента, например, <Button after={} /> ➡️ <Button.After /> – в этом случае мы должны позаботиться о синхронизации названий.

⚠️ IMPORTANT 2

Исключением являются логические свойства под RTL. С @VKCOM/vkui-design договорились переводить следующим образом:

Свойства

Значения

  • 1 кейс. Если речь про положение в одной плоскости.
    • leftstart
    • rightend
    • ❓(решается) horizontalinline или row
    • ❓(решается) verticalblock или column
  • 2 кейс. Если речь про координаты
    • topblock-start
    • rightinline-end
    • bottomblock-end
    • leftinline-start

Дизайн по возможности должен переименовать в Figma.

⚠️ IMPORTANT 3

В коде значения размерностей 's' | 'm' | 'l'. Дизайн может оставить полные названия.

flowchart TD
    A[Нашли расхождение в именовании] -->
    B{"Это особенность реализации в React?<br><br>Например, использование под-компонента или размерность (см. #6474) или логического синонима названия или антоним названия свойства (ex, в дизайне <code>enable</code>, в коде <code>disable</code>)"}
    B -->|Да| D["Пропускаем<br><br>По возможности смотрим можем ли добавить комментарий в документации"]
    B -->|Нет| E{"Справшиваем у дизайна могут ли они переименовать в Figma?<br><br>"}
    E -->|Да| F["Дизайн решает проблему у себя"]
    E -->|"Нет<br><br>либо техническая особенность Figma, либо iOS/Android разработка уже завязалась на текущее именование"| G["Добавляем в список в этой Epic задаче"]
Loading

Текущие задачи

@inomdzhon inomdzhon added this to the v7.0.0 milestone Sep 25, 2024
@inomdzhon inomdzhon added this to VKUI Sep 25, 2024
@github-project-automation github-project-automation bot moved this to 🗃 Backlog in VKUI Sep 25, 2024
@EldarMuhamethanov EldarMuhamethanov self-assigned this Sep 25, 2024
@inomdzhon inomdzhon modified the milestones: v7.0.0, v7.0.0-beta.1 Oct 4, 2024
@Kritolan
Copy link

Kritolan commented Oct 15, 2024

SimpleCell: переименовать expandable -> chevrone
MiniInfoCell: переименовать expandable → chevron

А почему разные итоговые имена?

@EldarMuhamethanov
Copy link
Contributor

А почему разные итоговые имена?

Опечатка, поправил

@EldarMuhamethanov
Copy link
Contributor

На данный момент все пункты выполнены - задачу закрываю

@github-project-automation github-project-automation bot moved this from 🔧 In progress to ✅ Done in VKUI Oct 22, 2024
@vkcom-publisher
Copy link
Contributor

v7.0.0-beta.1 🎉

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

No branches or pull requests

4 participants