1849010 - [css-fonts-4][css-fonts-5] Add font-synthesis-position CSS property

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, enhancement)

Description

[Mozilla Apprentice]

A resolution was made for csswg-drafts/#7441.

[css-fonts-4][css-fonts-5] Add "font-synthesis: super"

• RESOLVED: Add font-synthesis-position as defined in https://github.com/w3c/csswg-drafts/issues/7441#issuecomment-1444461111 except 'auto' means synthesis is required (not merely allowed)

Discussion.

Comment 1

[Jonathan Kew [:jfkthame]]

Attachment: Bug 1849010 - Implement the font-synthesis-position property. r=#layout-reviewers

Comment 2

[Jonathan Kew [:jfkthame]]

Just for the record, a PR to update the spec is at https://github.com/w3c/csswg-drafts/pull/9203.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/66c374acc3fb
Implement the font-synthesis-position property. r=devtools-reviewers,emilio

Comment 4

[Cosmin Sabou [:CosminS]]

Backed out for causing font-synthesis failures.

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&selectedTaskRun=JOnswB09TI6TzbNithWEkQ.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cusercancel&revision=66c374acc3fb80c145ded24b23fb3c24482e7c95

Failure log: https://treeherder.mozilla.org/logviewer?job_id=426358164&repo=autoland

Backout link: https://hg.mozilla.org/integration/autoland/rev/01b2cb7a4a75ee76d69b0eef1f7eae96e79de8b6

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41515 for changes under testing/web-platform/tests

Comment 6

[Jonathan Kew [:jfkthame]]

Ugh, forgot the animation-type list. (Seems a shame that isn't automatically generated from the property definitions...)

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fd08fa62bea0
Implement the font-synthesis-position property. r=devtools-reviewers,emilio

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 9

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/fd08fa62bea0

Comment 10

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 11

[Dave Letorey]

Unless I am missing something, I feel that this should work for <sub> & <sup> element as well as those with the property font-variant-position with the values of sub & super.

I have codepen with both these examples and it is not working as expected.

It works this way for font-variant-weight - <strong> & font-variant-style - <em>

Comment 12

[Jonathan Kew [:jfkthame]]

(In reply to Dave Letorey from comment #11)

No, because the <sub> and <sup> elements are not defined as using font-variant-position at all; they use font-size and vertical-align instead. See for example https://www.w3.org/TR/CSS22/sample.html.

(In many cases, font-variant-position might appear preferable, but the trouble is that these elements can potentially be nested, and font-variant-position wouldn't handle that.)

Comment 13

[Dave Letorey]

(In reply to Jonathan Kew [:jfkthame] from comment #12)

(In reply to Dave Letorey from comment #11)

No, because the <sub> and <sup> elements are not defined as using font-variant-position at all; they use font-size and vertical-align instead. See for example https://www.w3.org/TR/CSS22/sample.html.

(In many cases, font-variant-position might appear preferable, but the trouble is that these elements can potentially be nested, and font-variant-position wouldn't handle that.)

Thank you that, now, makes total sense.

Comment 14

[Dave Letorey]

Associated Pull Requests

• Content PR
• BCD PR
• interactive-examples PR
• data PR