-
Notifications
You must be signed in to change notification settings - Fork 707
[css-fonts-4] Angle for oblique->normal threshold should be lower end of common obliques not 20 #2539
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
Comments
Sure - because of #2295 the 20deg in this algo should be 14deg, anyway. |
It should be lower than that, is my argument. |
The Working Group just discussed
The full IRC log of that discussion<fantasai> Topic: Angle for oblique->normal threshold should be lower end of common obliques not 20<fantasai> https://github.com//issues/2539 <fantasai> github: https://github.com//issues/2539 <frremy> fantasai: in the font matching algorithm, we start searching from 20deg then go up from there <frremy> fantasai: 20deg is an average <frremy> fantasai: we should probably search from the minimum value <frremy> chris_: if the mimimum is 14 we should start searching at 7 <fantasai> "If no match is found, oblique values greater than or equal to 20deg are checked in ascending order followed by oblique values below 20deg in descending order, until 0 is hit." <frremy> fantasai: the spec says: <frremy> fantasai: (see above comment) <frremy> florian: 14 not 7? <frremy> myles: if you have 13deg and 80deg you want 13deg <frremy> fantasai: so we set 7deg <frremy> myles: but what about 6deg and 80deg? <frremy> florian: binary search, we look in both directions? <frremy> myles: I don't want to change the algorithm <frremy> myles: I think we should set the value as 11deg <frremy> myles: because this seems better than 8deg <frremy> fantasai: 11 is a lower-bound italic <frremy> fantasai: 8 would be pushing it <fantasai> s/italic/of typical italics/ <frremy> astearns: alright, any objection to start searching at 11 and see how it goes? <frremy> myles: (disgression on the negative angle) <chris_> It goes to 11! <frremy> RESOLVED: Start searching for obique starting from 11deg <florian> It goes to 11! -> it starts from 11! |
Hmm I see the spec still uses 20 deg in the |
And just checking (since there are 8 occurrences of Edit: including the part where italic is not found so you fall back to oblique. |
Also does this common scale mapping need to be changed?
|
All occurrences of 20deg related to oblique, so in the end they were all changed to 11. |
The font matching algorithm searches upward from the specified angle to more slanted fonts if the angle is > 20deg, and downward towards normal fonts if it is < 20deg. This threshold should not be the most common angle of obliques, but rather the lower end of that range, otherwise requests for an oblique angle which is reasonably oblique but not so much as the average will end up with a normal font.
The text was updated successfully, but these errors were encountered: