-
Notifications
You must be signed in to change notification settings - Fork 172
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
SGM-6973 - Clean and update typography sitewide #7014
Conversation
✅ Deploy Preview for sourcegraph ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Beautiful job! Left a couple of comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra comments
Thanks @traceyljohnson |
This PR was created by GitStart to address the requirements from this ticket: SGM-6973.
This ticket was imported from: SGM-6973
Description:
Updates the site-wide typography according to the Figma link:
https://www.figma.com/design/o1QRtdQI0ozKq0n7ATrKlx/Marketing-DLS?node-id=8435-37157&t=O1E8a0nW4Zk4jl5Y-0
DEMO:
https://www.loom.com/share/908c184f87b742f096bba69e9b5fa5da?sid=48121504-c679-4a93-94c5-9b8f36def212
Test cases:
Old font families Source Sans, system fonts are removed
Update Tailwind font families to 'sans': Inter 'mono': Roboto mono 'display': Space Grotesk.
Letter spacing matches letter spacing on figma
Updated blog template type styles (H1, H2, H3) and added Callout according to the specs
Test plan: