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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem with word rendering on Firefox and Edge with Jura font, font-weight 700 #229

Closed
HonkingGoose opened this issue May 23, 2020 · 4 comments
Labels
bug Code/Deployment issues

Comments

@HonkingGoose
Copy link
Contributor

oil-refining-word-refining-is-crushed

The string "Oil refining" in the <h5> header is rendered wrong.
The i blends in with the preceding letter f. It almost looks as if the i is hiding behind the f. 馃槃

I think the_Jura_ sans-serif font that's used for the <h5> header is causing this. When I toggle the Jura font off for the <h5> in Firefox Inspector, and use the default Times New Roman font the problem goes away.

The issue is also way better (but not completely fixed) when the font-weight is changed from 700 to 500.

I've checked the Jura font on Google fonts, and it has the same problem there with the string "Oil refining", but with black text on white background the problem is less noticeable, than with orange on black.

https://fonts.google.com/specimen/Jura?preview.text=Oil+refining&preview.text_type=custom

I'm using the Firefox browser on Windows 10.

@deniszholob
Copy link
Owner

Interesting, chrome is ok, but ff and edge do that...ughh, maybe ill pick a new font later -_-

@deniszholob deniszholob added the bug Code/Deployment issues label May 23, 2020
@HonkingGoose
Copy link
Contributor Author

Yeah browser display differences are no fun to deal with.

Just wanted to put the bug out there so that you're aware of it. Sorry for getting you extra work! 馃槃

@HonkingGoose HonkingGoose changed the title Problem with word rendering with Jura font, font-weight 700 Problem with word rendering on Firefox and Edge with Jura font, font-weight 700 May 23, 2020
@deniszholob
Copy link
Owner

NP, I appreciate all the findings! I can figure out the priority of things to fix and whatnot haha.

@pandalion
Copy link
Contributor

Hi, I have found the issue. You are including a link to Jura (<link href="https://fonts.googleapis.com/css?family=Jura" rel="stylesheet" type="text/css">) , but if you click on this link you will see it only includes the 400 weight of the font.

When you have bold headings, the browser is just guessing and making the webfont bold itself, but this can cause issues, such as the one you are seeing in Firefox. To fix this, we just need to specify the other weights of Jura to be loaded, and then the webfont for weight 700 will be used :) e.g.

<link href="https://fonts.googleapis.com/css2?family=Jura:wght@400;700" rel="stylesheet" type="text/css">

I will make a PR for the fix now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Code/Deployment issues
Projects
None yet
Development

No branches or pull requests

3 participants