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

M3-5546: IP address font color on Firefox in dark mode #8008

Merged
merged 2 commits into from
Oct 27, 2021

Conversation

tiffwong
Copy link
Contributor

Description

Previously, if you go to any Linode Detail page on Firefox in dark mode, the font color for "IP Addresses" and "SSH Access" should be white but instead, it's showing up as black.

This is due to a known Firefox bug that will decide that a font is colored if it contains SVG glyphs (which surprise, Source Code Pro does). The latest release should've fixed this issue but the font is still showing up as black for me after updating the files.

After some sleuthing, I decided to test out Ubuntu Monospace as an alternative to Source Code Pro. I tested the font color on Chrome, Firefox, Safari, Edge, Opera, and Vivaldi with no issues.

Links:

How to test

  • On develop, use Firefox and go to any Linode Detail page in dark mode to verify the problem
  • The font color should now be white after checking out this PR

@tiffwong tiffwong added the UX/UI Changes for UI/UX to review label Oct 22, 2021
@tiffwong tiffwong self-assigned this Oct 22, 2021
Copy link
Contributor

@DevDW DevDW left a comment

Choose a reason for hiding this comment

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

Everything looks good to me in both Firefox and Chrome now, nice work 🚢

Copy link
Contributor

@HanaXu HanaXu left a comment

Choose a reason for hiding this comment

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

LGTM in Chrome, Firefox, and Safari. Great job! 🚀

@tiffwong tiffwong merged commit 2a218c2 into linode:develop Oct 27, 2021
@tiffwong tiffwong deleted the M3-5546-firefox-ip-text branch October 27, 2021 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UX/UI Changes for UI/UX to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants