Closed
Bug 1447865
Opened 6 years ago
Closed 5 years ago
Implement font-variant descriptor for @font-face rules [was: interaction of font-variant descriptor, font-variant-ligatures property, font-variant shorthand property & font-feature-settings low-level property for ligatures is incorrect]
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: chris, Unassigned)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [gfx-noted])
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:61.0) Gecko/20100101 Firefox/61.0 Build ID: 20180321102906 Steps to reproduce: Run https://test.csswg.org/harness/test/css-fonts-3_dev/single/font-variant-05/format/html5/ Actual results: Top three lines have crosses instead of check marks for the last two positions. This test is using a special font to check which OpenType features are enabled or disabled. Expected results: All lines should look identical. Setting ‘font-variant-ligatures’ properties will override the same properties set in the @font-face rule. But properties set there and not explicitly turned off or on later remain set. Tests interaction of font-variant descriptor, font-variant-ligatures property, font-variant shorthand property & font-feature-settings low-level property for ligatures, in accordance with steps 1,2,4 and 6 of section 7.2 Feature precedence https://www.w3.org/TR/css-fonts-3/#feature-precedence tested in Beta 60 and Nightly 61 on Windows 10
Comment 1•6 years ago
|
||
Needinfo'ing jfkthame here as I'm not sure what needs to be done here, or what plans are.
Flags: needinfo?(jfkthame)
Whiteboard: [gfx-noted]
Comment 2•6 years ago
|
||
AFAICT, Gecko has never implemented font-variant as a descriptor in @font-face rules (it only exists as a CSS property). It can hardly "interact correctly" when it doesn't exist! :( Looks like something we just omitted to do, back when font feature support was implemented -- and AFAIK nobody has reported it until now.
Status: UNCONFIRMED → NEW
Component: Graphics: Text → CSS Parsing and Computation
Ever confirmed: true
Flags: needinfo?(jfkthame)
Summary: interaction of font-variant descriptor, font-variant-ligatures property, font-variant shorthand property & font-feature-settings low-level property for ligatures is incorrect → [was: interaction of font-variant descriptor, font-variant-ligatures property, font-variant shorthand property & font-feature-settings low-level property for ligatures is incorrect
Updated•6 years ago
|
Summary: [was: interaction of font-variant descriptor, font-variant-ligatures property, font-variant shorthand property & font-feature-settings low-level property for ligatures is incorrect → Implement font-variant descriptor for @font-face rules [was: interaction of font-variant descriptor, font-variant-ligatures property, font-variant shorthand property & font-feature-settings low-level property for ligatures is incorrect]
Comment 3•6 years ago
|
||
Chris, do you know if any UA actually implements the font-variant descriptor?
Flags: needinfo?(chris)
I have made a PR on web platform tests to check for that https://github.com/w3c/web-platform-tests/pull/10415 From initial tests, Safari does implement the descriptor, while Chrome and Gecko do not. If we can get a second implementation this will stay in CSS Fonts 3, otherwise it will get punted to Fonts 4. Being able to set your preferred default values in a descriptor, which gets applied wherever that font-family is used, is handy. You can override specific ones with the properties, of course.
Flags: needinfo?(chris)
Updated•6 years ago
|
Priority: -- → P3
Updated•6 years ago
|
Blocks: css-fonts-3
Updated•6 years ago
|
Updated•6 years ago
|
Keywords: dev-doc-needed
Comment 6•5 years ago
|
||
Per https://github.com/w3c/csswg-drafts/issues/2531 this is WONTFIX. Bug 1581729 is for removing the partial support for this (such as FontFace.variant
).
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
Updated•5 years ago
|
Keywords: dev-doc-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•