Closed Bug 1849010 Opened 7 months ago Closed 7 months ago

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

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
118 Branch
Tracking Status
firefox118 --- fixed

People

(Reporter: mozilla-apprentice, Assigned: jfkthame)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

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

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

Discussion.

Component: Layout → Layout: Text and Fonts
Type: defect → enhancement
Summary: [css-fonts-4][css-fonts-5] Add "font-synthesis: super" → [css-fonts-4][css-fonts-5] Add font-synthesis-position CSS property
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED

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

Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/66c374acc3fb
Implement the font-synthesis-position property. r=devtools-reviewers,emilio
Keywords: dev-doc-needed
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41515 for changes under testing/web-platform/tests

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

Flags: needinfo?(jfkthame)
Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fd08fa62bea0
Implement the font-synthesis-position property. r=devtools-reviewers,emilio
Upstream PR was closed without merging
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch
Upstream PR merged by moz-wptsync-bot

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>

Flags: needinfo?(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.)

Flags: needinfo?(jfkthame)

(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.

You need to log in before you can comment on or make changes to this bug.