Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork66
Bugfix: quote monospace font#85
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
For some reason, saying just `monospace` here doesn't seem to work on chrome mobile. Quoting monospace fixes the issue.
Seehttps://twitter.com/sadhlife/status/1446060335979397120 for the sleuthing :) |
Good work! As it happens, this is fixed in the upcoming Chrome 96:python/pythondotorg#1708 (comment) |
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.
Despite the upcoming fix in Chrome, this is such a simple fix, I think it's worth merging for people who will still be on Chrome < 96, even after Chrome 96 release.
(Plus for people using latest Chrome 94 now :)
Oh this kind of bugs … ☹ |
Thanks@tusharsadhwani and@nedbat! |
I released it as python-docs-theme 2021-11 (I live in the future) and rebuilthttps://docs.python.org/3/ so someone can check. Rebuild for other versions and languages will be automatically done in the next 24h. |
beauremus commentedOct 7, 2021
This "fix" broke the font for me. My Chrome on my Mac is now loading Helvetica for code rather than Courier. I can confirm that removing the quotes from |
Chrome, showing 3 different behaviours on 3 operating systems. A tale as old as time. |
Perhaps the best fix is to just specify monospace. Why a fallback to sans-serif? Does any browser not have a monospace font that is used when you specify monospace? |
Removing the quotes from monospace even with it being the only specified font breaks on current chrome mobile. |
So... Do we need one monospace with quotes and one monospace without quotes? 🙃
The problem seems to be desktop isn't picking up the "monospace" and falling back to sans-serif. If we remove sans-serif, it'll fall back to some system default, which will be some sans-serif or serif, but not monospace. |
Okay, I tested on 3.9's page, and it seems just pre {font-family: monospace;} seems to work. Can someone test it on all 3 platforms, android mac and windows, that'd be great. |
arseniiv commentedOct 7, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Noticed my Firefox 93.0 on win10 is weirded by |
brandon-leapyear commentedOct 7, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✨This is an old work account. Please reference@brandonchinn178 for all future communication ✨ Can confirm, currently fonts on |
tusharsadhwani commentedOct 7, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Can confirm, works fine. Even on mobile where just |
There is (or was) a known problem with plain Andnecolas/normalize.css#519 (comment) suggests a reason, browsers looked for the fixed string |
Ah, that makes sense now. So I guess |
beauremus commentedOct 7, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@beauremus Thanks a bunch. |
Uh oh!
There was an error while loading.Please reload this page.
For some reason, saying just
monospace
here doesn't seem to work on chrome mobile. Quoting monospace fixes the issue.Before:

After:
