Skip to content

remove google fonts#2721

Merged
CollinBeczak merged 2 commits into
mainfrom
remove-google-fonts
Sep 4, 2025
Merged

remove google fonts#2721
CollinBeczak merged 2 commits into
mainfrom
remove-google-fonts

Conversation

@CollinBeczak

Copy link
Copy Markdown
Collaborator

Possible solution to #2720

@jake-low

jake-low commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

I'm all for removing Google Fonts.

This PR makes MapRoulette use the default system-ui font. This looks fine to me on Firefox + macOS (see below). Are you by any chance able to test this on any other platforms, to make sure nothing breaks?

Note: you may also want to change the font stack in tailwind.config.js to remove Roboto. In fact, that list could be simplified to just ["system-ui", "sans-serif"].

With Roboto (loaded from Google Fonts)

image image

With default system fonts

image image

@CollinBeczak

Copy link
Copy Markdown
Collaborator Author

Tested on chrome, and firefox... i can check on safari

@jake-low

jake-low commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

Tested on chrome, and firefox... i can check on safari

Sorry, should have clarified: I'm mainly concerned about different operating systems (Ubuntu, Windows, etc) which have their own sets of default fonts installed. I don't have machines running either of those OSes so I can't easily test this there. It should work fine though, and I'm fine merging this without specifically testing those. Simplifying the font stack (as you've done) also makes this change less likely to break the UI on systems we haven't tested.

@CollinBeczak CollinBeczak merged commit 70daad8 into main Sep 4, 2025
6 checks passed
@CollinBeczak CollinBeczak deleted the remove-google-fonts branch September 4, 2025 17:45
@CollinBeczak CollinBeczak mentioned this pull request Sep 10, 2025
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.

2 participants