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

Feat/figma theme #7

Merged
merged 3 commits into from
May 7, 2024
Merged

Feat/figma theme #7

merged 3 commits into from
May 7, 2024

Conversation

kirill-ivanovvv
Copy link
Member

@kirill-ivanovvv kirill-ivanovvv commented May 6, 2024

Closes #1

Вопросы.

  • огромное количество теней, в файле с тенями, это нормально? в макете только один стиль.
  • не все текстовые стили нашел. в фигме вижу 39й размер, в файлах такого нет.
  • в фигме вижу только шрифт - Inter, в файлах его нет.

Как работает этот скрипт? Он проходится именно по стилям в макете или бегает по всему макету?
Я сравнивал итоговые данные именно со стилями.
P.S. Под стилями имеюю ввиду данные с панели справа.

Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Все верно получил.

Теперь сделай провайдер темы. Можешь подсмотреть в hyperion либо в других наших проектах.

@Nelfimov
Copy link
Member

Nelfimov commented May 6, 2024

  • огромное количество теней, в файле с тенями, это нормально? в макете только один стиль.
    Вероятно нет. Можешь лишние убрать - по ходу работы над проектом нужные добавим.
  • не все текстовые стили нашел. в фигме вижу 39й размер, в файлах такого нет.
    Можешь вручную добавить сейчас либо по ходу разработки.
  • в фигме вижу только шрифт - Inter, в файлах его нет.

Можно вручную добавить.

Как работает этот скрипт? Он проходится именно по стилям в макете или бегает по всему макету? Я сравнивал итоговые данные именно со стилями. P.S. Под стилями имеюю ввиду данные с панели справа.

99% что бегает по всему макету. Это область для доработки.

@kirill-ivanovvv
Copy link
Member Author

запушил коммит и забыл отформатировать yarn check
на локальном репозитории поправил и сделал git --amend

коммиты слить не получилось, как делать лучше в таких случаях?

@Nelfimov
Copy link
Member

Nelfimov commented May 6, 2024

squash и force push. До ревью форс пуш - можно.

feat: theme-provider
@Nelfimov Nelfimov merged commit 071394b into master May 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Генерация темы
2 participants