search

atls / team-management

Team management app
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Feat/figma theme #7

Closed effektivnayarabota1 closed 4 months ago

effektivnayarabota1 commented 4 months ago

Closes https://github.com/atls/team-management/issues/1

Вопросы.

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

Nelfimov commented 4 months ago

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

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

  • в фигме вижу только шрифт - Inter, в файлах его нет.

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

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

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

effektivnayarabota1 commented 4 months ago

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

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

Nelfimov commented 4 months ago

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