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

CSS Font shorthand gets populated incorrectly #224

Closed
MDSLKTR opened this issue Jun 6, 2018 · 3 comments
Closed

CSS Font shorthand gets populated incorrectly #224

MDSLKTR opened this issue Jun 6, 2018 · 3 comments

Comments

@MDSLKTR
Copy link

MDSLKTR commented Jun 6, 2018

Declaration:
@font-face {
font-family: 'PorscheNextWLa';
src: url('../fonts/PorscheNextWLa-Regular.woff') format('woff');
font-weight: normal;
font-style: normal;
}

Also there is 'font-size: 100%' property on the body which might be incorrectly merged with the rem font-size of the heading. Nonetheless if i remove it the problem persists, so it might be something else. Chrome discards the whole rule if you supply two values for font size.

Result:
bildschirmfoto 2018-06-06 um 14 08 47

If I remove the 100% from the 'font: 400 100% 54px/54px PorscheNextWlaBold' declaration it works as expected and the font is applied. It seems the CSS font shorthand is incorrectly filled.

Expected behavior

Webfont is properly inlined and font declarations are working

Actual behavior (stack traces, console logs etc)

Headlines use Fallback fonts because the font declaration in CSS in wrong.

Library version

2.5.2

Browsers

Chrome 67

@jwareing
Copy link

jwareing commented Jun 6, 2018

Check out #220

@shayneslitehouseinc
Copy link

Can the rendering filter of style override this?

@MDSLKTR
Copy link
Author

MDSLKTR commented Jun 9, 2018

@shayneslitehouseinc I don't think so, this is something Chrome does internally when collapsing font-* properties. But you could try.

@MDSLKTR MDSLKTR closed this as completed Nov 13, 2023
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

No branches or pull requests

3 participants