Hi Philip, I noticed that font-size-adjust-012.html, font-size-adjust-013.html, and parts of font-size-adjust-computed.html (e.g., ch-width and ic-width) show minor deviations from expected results. The exact issue with the WPT Linux machine is unclear since these tests can be affected by various font rendering configuration and environment (e.g., hinting, sub-pixel rendering, fonts installed, font rendering libraries, etc.). My Linux machine and the Linux try bots pass these tests. I think syncing the WPT Linux machine's font setup with our Linux trybot's setup could help.
The only actual failure I know of is the ic-height test of font-size-adjust-computed.html. This is due to unclear fallback behavior in the font spec when the necessary font metrics are missing, which is still under discussion by CSSWG, as I noted. Depending on the resolution, some test results can change later. I will follow up on this standardization effort. Best, -- ChangSeok > On May 21, 2024, at 5:43 AM, Philip Jägenstedt <foo...@chromium.org> wrote: > > Hi ChangSeok, > > Thank you for working on this, it's great to see both more powerful > typography control, and progress on Interop 2024. > > Can you say something about the remaining failures in WPT? > font-size-adjust-012.html font-size-adjust-013.html both look like minor > differences, but it seems like it only happens on Linux (Chrome) and not > Windows (Edge)? > > There are also 3 failing subtests in font-size-adjust-computed.html, does > that depend on resolving one of the spec issues? Or can we match what Firefox > and Safari do and pass the tests without waiting for spec changes? > > Best regards, > Philip > > On Thu, May 16, 2024 at 5:01 PM Vladimir Levin <vmp...@chromium.org> wrote: > > > On Thu, May 16, 2024 at 10:02 AM ChangSeok Oh <changs...@chromium.org> wrote: > Contact emails > changseok...@bytedance.com, changs...@chromium.org > > Explainer > None > > Specification > https://www.w3.org/TR/css-fonts-5/#font-size-adjust-prop > > Summary > The font-size-adjust CSS property enhances readability consistency by > adjusting font size based on lowercase letter height rather than uppercase. > Additionally, the newly introduced two-value syntax for font-size-adjust in > the font module level 5 enables web designers to specify a font metric for > size adjustment. This feature is one of focus areas for Interop 2024. > > Blink component > Blink>CSS, Blink>Fonts > > TAG review > None > > TAG review status > Not applicable > > Risks > Interoperability and Compatibility > Gecko and WebKit have shipped this feature, and there is no major > interoperability risk. However, certain aspects are still being discussed > regarding interoperability concerns. [1, 2]. > [1] https://github.com/w3c/csswg-drafts/issues/6384 [2] > https://github.com/w3c/csswg-drafts/issues/10292 > > Gecko: Shipped/Shipping (https://bugzilla.mozilla.org/show_bug.cgi?id=1711479) > > WebKit: Shipped/Shipping (https://bugs.webkit.org/show_bug.cgi?id=254191) > > Web developers: No signals > > Other signals: > > WebView application risks > Does this intent deprecate or change behavior of existing APIs, such that it > has potentially high risk for Android WebView-based applications? > None > > Debuggability > N/A > > Will this feature be supported on all six Blink platforms (Windows, Mac, > Linux, ChromeOS, Android, and Android WebView)? > Yes. Windows experiences a minor sub-pixel mismatching issue in one wpt test, > but it doesn't affect usability. > > Is this feature fully tested by web-platform-tests? > Yes > https://wpt.fyi/results/css/css-fonts?label=master&label=experimental&aligned&view=interop&q=label%3Ainterop-2024-font-size-adjust > > Flag name on chrome://flags > enable-experimental-web-platform-features > > Finch feature name > CSSFontSizeAdjust > > Requires code in //chrome? > False > > Tracking bug > - https://issues.chromium.org/issues/40081245 > - https://issues.chromium.org/issues/40186237 > > Measurement > - https://chromestatus.com/metrics/css/timeline/popularity/465 > - https://caniuse.com/font-size-adjust > > Availability expectation > Both the Gecko and WebKit communities have implemented this feature, and > corresponding tests are already included in the Web Platform Tests (WPT). > > Adoption expectation > Firefox and WebKit variant browsers have embraced this feature. > > Adoption plan > Blink is shipping this feature. > > Non-OSS dependencies > Does the feature depend on any code or APIs outside the Chromium open source > repository and its open-source dependencies to function? > None > > Sample links > https://developer.mozilla.org/en-US/docs/Web/CSS/font-size-adjust > > Estimated milestones > Shipping on desktop > 127 > Shipping on Android > 127 > Shipping on WebView > 127 > > Anticipated spec changes > Open questions about a feature may be a source of future web compat or > interop issues. Please list open issues (e.g. links to known github issues in > the project for the feature specification) whose resolution may introduce web > compat/interop risk (e.g., changing to naming or structure of the API in a > non-backward-compatible way). > > Some concerns related to interoperability are still under discussion. > - https://github.com/w3c/csswg-drafts/issues/6384 > - https://github.com/w3c/csswg-drafts/issues/10292 > > Link to entry on the Chrome Platform Status > - https://chromestatus.com/feature/5720910061371392 > - https://chromestatus.com/feature/5170570175447040 > > Do you mind starting the various reviews (including API Owners) in the > chromestatus entry?: > > <chips.png> > > Thanks! > > > > Links to previous Intent discussions > - Intent to prototype font-size-adjust: > https://groups.google.com/a/chromium.org/g/blink-dev/c/lqd_g6Z6fH4/m/qZyljJmg6oQJ > - Intent to prototype the two value syntax: > https://groups.google.com/a/chromium.org/g/blink-dev/c/2FufB-ckycw/m/mj1I3iUECAAJ > > Best, > > -- > You received this message because you are subscribed to the Google Groups > "blink-dev" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to blink-dev+unsubscr...@chromium.org. > To view this discussion on the web visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/b34b218a-28c4-44e0-9451-3a2fc0949ab3n%40chromium.org. > > -- > You received this message because you are subscribed to the Google Groups > "blink-dev" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to blink-dev+unsubscr...@chromium.org. > To view this discussion on the web visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CADsXd2NOwQS9mwe7rrShcyapRmWTFh6%3DUTo9%2BGE%2B2E525TrpEA%40mail.gmail.com. -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/7768618E-0A7B-4093-95E0-D20555AB87E5%40chromium.org.
