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(Gradient): add new gradient color #582

Merged
merged 5 commits into from
Sep 12, 2023
Merged

Conversation

BlackySoul
Copy link
Contributor

Перед отправкой этого реквеста на ревью убедитесь, что:

  • в вашей ветке работает сборка (npm run build:local),
  • в вашей ветке проходят тесты (npm test),
  • в вашей ветке проходит линтер (npm run lint), некоторые ошибки можно автоматически поправить с помощью npm run lint:fix,
  • покрытие тестов не ниже минимального значения,
  • если вы вносите изменения в задокументированные части библиотеки,
    ваш pull request содержит обновления документации,
  • если вы вносите изменения в сами токены, вы
    согласовали их с дизайнерами.

Подробнее о внесении изменений в репозиторий токенов

@BlackySoul BlackySoul requested a review from a team as a code owner August 28, 2023 11:16
@BlackySoul BlackySoul self-assigned this Aug 28, 2023
@Fliqle
Copy link
Contributor

Fliqle commented Aug 29, 2023

@BlackySoul, а зачем мы делаем этот хук в теме Paradigm, и почему не делаем такого в теме VK?

@BlackySoul
Copy link
Contributor Author

@BlackySoul, а зачем мы делаем этот хук в теме Paradigm, и почему не делаем такого в теме VK?

Какой именно хук?)

@Fliqle
Copy link
Contributor

Fliqle commented Aug 29, 2023

@BlackySoul, а зачем мы делаем этот хук в теме Paradigm, и почему не делаем такого в теме VK?

Какой именно хук?)

C colorBackgroundContentDark

@BlackySoul
Copy link
Contributor Author

@BlackySoul, а зачем мы делаем этот хук в теме Paradigm, и почему не делаем такого в теме VK?

Какой именно хук?)

C colorBackgroundContentDark

Просто вынесла определение цвета в переменную

В VK немного по-другому формируются токены (берутся из json'a с цветами из фигмы), поэтому там переменная не нужна

Сейчас ещё обратила внимание, что в VK теме по какой-то причине остались вынесены два цвета, хотя они тоже должны браться из токенов, узнаю, поправлю

@BlackySoul
Copy link
Contributor Author

@Fliqle обновила

Fliqle
Fliqle previously approved these changes Aug 29, 2023
@github-actions
Copy link

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
91.66% (-0.01% 🔻)
1143/1247
🟢 Branches 88.89% 320/360
🟢 Functions 92.86% 169/182
🟢 Lines
91.31% (-0.01% 🔻)
1061/1162

Test suite run success

392 tests passing in 37 suites.

Report generated by 🧪jest coverage report action from 675325c

@8coon 8coon changed the base branch from master to 4.39.0 September 12, 2023 11:40
@8coon 8coon merged commit 6baea7e into 4.39.0 Sep 12, 2023
5 checks passed
@8coon 8coon mentioned this pull request Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants