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

sans-serif backup for system-ui (fix #630) #631

Merged
merged 5 commits into from
Jun 26, 2024

Conversation

hchiam
Copy link
Contributor

@hchiam hchiam commented Jun 26, 2024

fix #630 - give system-ui font fallbacks (including sans-serif) like label.element.css does, based on this comment

instead of only appending sans-serif

below are screenshots of the 5 areas corresponding to the 5 edits, if someone can test how it looks on their machine before and after


1 of 5: distance element <figcaption>:


2 of 5: metatip element <figure>:


3 of 5: metatip element [longform]:


4 of 5: vis-bug element hover tooltip inherits font-family from ol:


5 of 5: base element hotkeys <hotkeys-move>:

Copy link
Contributor Author

@hchiam hchiam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ready for review.

Copy link
Member

@argyleink argyleink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rad ty @hchiam!

@argyleink argyleink merged commit 8efc01e into GoogleChromeLabs:main Jun 26, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Linux system-ui missing, needs sans-serif backuip
2 participants