-
Notifications
You must be signed in to change notification settings - Fork 73
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
[BD-46] feat: update design tokens #1748
[BD-46] feat: update design tokens #1748
Conversation
Thanks for the pull request, @PKulkoRaccoonGang! When this pull request is ready, tag your edX technical lead. |
✅ Deploy Preview for paragon-openedx ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
36cd214
to
f3eb5e3
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## feat/design-tokens #1748 +/- ##
=====================================================
Coverage ? 90.51%
=====================================================
Files ? 211
Lines ? 3713
Branches ? 872
=====================================================
Hits ? 3361
Misses ? 337
Partials ? 15 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
47b98af
to
5167c37
Compare
1e0a61f
to
e79f6c1
Compare
e79f6c1
to
8d8054f
Compare
8d8054f
to
fd6d164
Compare
2e25efa
to
cd21040
Compare
94cab5d
to
46bdabe
Compare
46bdabe
to
a60b460
Compare
f6de564
to
0c79b76
Compare
@PKulkoRaccoonGang 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Description
modify
parameter in JSON files to apply SASS functions to CSS variables;map-get
function with{path.to.design.token.value}
in JSON files;subtract
andadd
functions withcalc
;replace-variables-definition
forscss/core/_variables.scss
file and try to fix issues with bootstrap.Deploy Preview
Paragon Design System
Merge Checklist
example
app?wittjeff
andadamstankiewicz
as reviewers on this PR.Post-merge Checklist