-
Notifications
You must be signed in to change notification settings - Fork 14
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: update typography as per UI Kit #301
Conversation
Branch preview✅ Deployed successfully in branch deployment: |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
7b94712
to
260347f
Compare
ESLint Summary View Full Report
Report generated by eslint-plus-action |
@TanyaEfremova can you review the typography changes across the pages? The font change is very noticeable so I'd like to have your 👍 before merging it into production. Link -> https://update_typography--homepage.review.5afe.dev/ |
Some of the headings use font weight light (300), however, it was only intended for the body text. |
You're right! Just fixed it. Can you check again? |
Update theme's typography as per latest UI Kit guide -> https://www.figma.com/file/qtXXlwdslAze6yNbLLzIRa/Website-redesign-2.0?type=design&node-id=10829-708&mode=design&t=zuBY8bCFKL1ILz9H-0
Typography feedback was collected here: https://www.notion.so/safe-global/Typography-Updates-QA-54b63bdeab744e80a5605402a85e7753
I've addressed in #368 the feedback I found unrelated with this change