-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Support font-optical-sizing
#40829
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
Support font-optical-sizing
#40829
Conversation
Signed-off-by: Simon Wülker <simon.wuelker@arcor.de>
177c67d to
4d55d75
Compare
|
🔨 Triggering try run (#19604170858) for Linux (WPT) |
|
It looks like a bunch of other styles (e.g. |
|
Test results for linux-wpt from try job (#19604170858): Flaky unexpected result (31)
Stable unexpected results that are known to be intermittent (21)
Stable unexpected results (5)
|
|
|
I'm currently working towards #37236 which doesn't require the font-variant- properties, but renders incorrectly without |
4d55d75 to
48618d2
Compare
|
🔨 Triggering try run (#19610010614) for Linux (WPT) |
|
Test results for linux-wpt from try job (#19610010614): Flaky unexpected result (49)
Stable unexpected results that are known to be intermittent (24)
|
|
✨ Try run (#19610010614) succeeded. |
Signed-off-by: Simon Wülker <simon.wuelker@arcor.de>
48618d2 to
e49439c
Compare
Signed-off-by: Simon Wülker <simon.wuelker@arcor.de>
|
The |
It should be valid, the two numbers specifiy a range ( |
This doesn't quite fix the issue in #38642 (comment), I think macos should still have the wrong behaviour for
font-optical-sizing: none(but I have no way to verify that).Manual testcase (fixed by this change)
Testing: We don't run wpt with
layout_variable_fontsenabled yet. (But I think we could after this change)Depends on servo/stylo#271
Part of #38800