-
-
Notifications
You must be signed in to change notification settings - Fork 37
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
HTML display issues in docs #46
Comments
Thanks! I hope you enjoy using Term! Where in the docs is that? It looks fine for me here: https://fedeclaudi.github.io/Term.jl/stable/ |
It's the first page of the docs: https://fedeclaudi.github.io/Term.jl/dev/ I'm on MacOS Monterey, latest Safari (15.1). It's the same on FireFox (98.0.1). Just checked, it's the same on my iPhone. Perhaps there's a different interpretation of CSS... 😱😱 |
mmm bizarre, it looks fine for me anywhere. Perhaps try ctrl+shift+R to reload? |
Whaaat, that's so weird. This is on the docs? |
It looks like this might be an issue with the JuliaMono typeface. If you swap to another monospace font, it lines up correctly: Screen.Recording.2022-03-21.at.2.08.51.PM.mp4On closer inspection, it looks like this only happens on lines with bolded areas. JuliaMono is supposed to be monospace regardless of weight, but apparently that's not happening here. I checked out the CSS stylesheet and it looks like only @font-face {
font-family: JuliaMono;
/* Pulling the font file straight from the JulaMono repo for convenience */
src: url("https://github.com/cormullion/juliamono/blob/master/webfonts/JuliaMono-Bold.woff2?raw=true") format("woff2");
font-weight: bold;
text-rendering: optimizeLegibility;
} then it works as intended (and looks really nice, great work on this fantastic package! 😍): If I were to take a guess, I'd think maybe the difference in appearance between users might be based on 1) how their browser handles faux-bolding, or perhaps 2) whether they actually have JuliaMono installed on their device already. Edit: This might be an upstream issue with Documenter.jl; I opened an issue there as well. |
Thank you, I hope you enjoy it! Since it's not a Term issue I'll close, unless you need to open to keep track of it but I see you've opened a related one. |
Yeah, go ahead and close it, and hopefully I can get it fixed soon on the Documenter.jl end. |
Thanks for the cool sleuthing, @mostol ! |
Just an update: A quick CSS change was merged into Documenter.jl (see above), so this should theoretically be cleared up whenever the Term.jl docs are built/published with the latest version including the fix. (Hats off to @cormullion for very conveniently already having the necessary CSS file hosted and ready-to-import.) |
By the way, is there any advice on how ti display stuff like progress bars? I also seem to have broken something at the bottom of this page: https://fedeclaudi.github.io/Term.jl/dev/basics/colors/ |
Nice package! I saw this in the documentation:
I don't know why the whitespace isn't being formatted correctly though. Perhaps it's how Documenter does it...
The text was updated successfully, but these errors were encountered: