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

Switch global font to Computer Modern #110

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jengelh
Copy link

@jengelh jengelh commented Mar 13, 2024

A sample PR for employing the same font in the HTML as is used in the PDF. There is a bit of concern though: Serif fonts and low-resolution (96dpi) displays don't historically mix well, which is why you don't see them all that often on webpages (e.g. newspaper sites). It's ok when you look at it with a mobile phone (just tested at 400dpi) or set browser page magnification to 150% on a low-dpi screen, so 🤷 goodenoughforme.

Fixes #98 .

Fixes #103 (no longer using font fragments with unicode-range).

Incidentally, this also fixes contacting Google services (GDPR and all that).

Haven't looked at the MathJAX parts at all(!), I just made sure the new CSS renders ok with https://eel.is/c++draft/format.string.escaped#example-1 .

This is the same font that is already used in the PDF version.
Adresses Eelis#98 .

This incidentally fixes
Eelis#103 too, because we
are no longer using font fragments with ``unicode-range``.

Incidentally, this *also* fixes contacting Google services (GDPR and
all that).
@Eelis
Copy link
Owner

Eelis commented Mar 16, 2024

This reduces the load time of /full on my machine from almost 3 minutes, to about 10 seconds, which is awesome!

But it doesn't solve the wideness and header overflow issue mentioned in #98, because it changes from the status quo:
image
to:
image
(In the PDF it looks like:)
image

@jengelh
Copy link
Author

jengelh commented Mar 16, 2024

overflow behavior is a separate topic to look at

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.

Courier New does not render U+0301 correctly [format.string.escaped] example is rendered incorrectly
2 participants