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

Convert slant angle from CSS to OpenType for web font instantiation #22357

Merged
merged 1 commit into from
Mar 20, 2020

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Mar 20, 2020

OpenType and CSS use opposite directions for the slant angle of a
variable font, compare note in [1] and test case in [2]. After the issue
was resolved in the CSS spec to keep these opposite directions, convert
from CSS to OpenType before shaping.

[1] https://drafts.csswg.org/css-fonts/#font-style-prop
[2] https://arrowtype.github.io/vf-slnt-test/

Bug: 859869
Change-Id: Ib1cc9224edab1dc17d8c6ed200e8369714cc73ff
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109762
Commit-Queue: Koji Ishii <[email protected]>
Reviewed-by: Koji Ishii <[email protected]>
Cr-Commit-Position: refs/heads/master@{#752096}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

@drott
Copy link
Contributor

drott commented Mar 20, 2020

CC @jfkthame, @litherum - see also #22355.

OpenType and CSS use opposite directions for the slant angle of a
variable font, compare note in [1] and test case in [2]. After the issue
was resolved in the CSS spec to keep these opposite directions, convert
from CSS to OpenType before shaping.

[1] https://drafts.csswg.org/css-fonts/#font-style-prop
[2] https://arrowtype.github.io/vf-slnt-test/

Bug: 859869
Change-Id: Ib1cc9224edab1dc17d8c6ed200e8369714cc73ff
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109762
Commit-Queue: Koji Ishii <[email protected]>
Reviewed-by: Koji Ishii <[email protected]>
Cr-Commit-Position: refs/heads/master@{#752096}
@chromium-wpt-export-bot chromium-wpt-export-bot merged commit c98c762 into master Mar 20, 2020
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-2109762 branch March 20, 2020 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants