FF150 Relnote: CSSFontFaceDescriptors#43621
Conversation
|
Preview URLs (3 pages)
External URLs (1)URL:
(comment last updated: 2026-04-17 04:10:04) |
|
This pull request has merge conflicts that must be resolved before it can be merged. |
07532be to
c7c510a
Compare
chrisdavidmills
left a comment
There was a problem hiding this comment.
@hamishwillee couple of bits for you to look at.
| #### Result | ||
|
|
||
| {{EmbedLiveSample("Accessing @font-face properties", "100%", "250px")}} | ||
|
|
There was a problem hiding this comment.
The live example looks fine. Just one thing — in the BCD table, what I assume is the enrty for this change is worded as "Type changed to CSSFontFaceDescriptors". It doesn't say supported by Fx150 yet, and the wording itself is a bit confusing because it implies that the interface's type property will return CSSFontFaceDescriptors, but actually it is the style property that will return that. It would be less confusing if it were phrased something like "style returns a CSSFontFaceDescriptors object."
I understand you are working on this, but I'm flagging it anyway just in case.
There was a problem hiding this comment.
@chrisdavidmills Thanks for the review. I've taken most suggestions. Also updated the style page too to make it a live example.
I don't plan to do anything about this, because it is a BCD issue.
- The table has since updated so it does say FF150
- The "Type changed to" is nested under style, and that is shown visually. I don't disagree that it isn't hugely obvious, but that's out of scope for me there.
43ce0b5 to
4ba3051
Compare
7a4ad8b to
08602f0
Compare
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
08602f0 to
007983c
Compare
|
@chrisdavidmills Can you look at this again please. |
chrisdavidmills
left a comment
There was a problem hiding this comment.
@hamishwillee, this is looking GTG. Merging!
FF150 adds support for
CSSFontFaceRule.stylebeing aCSSFontFaceDescriptors. This adds a release note and minor updates to docs.Related docs work can be tracked in #43564