> On May 23, 2024, at 10:19 AM, Vladimir Levin <vmp...@chromium.org> wrote: > > > > On Wed, May 22, 2024 at 4:00 PM ChangSeok Oh <changs...@chromium.org> wrote: > 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. > > One thing you can try is running this test from wpt.live on your local build. > In the past, I've used that to reproduce failures that would otherwise pass > on the bots.
This is a good idea worth trying. When I casually tested it with an existing Chromium build on Linux, I encountered a technical issue with running WPT. I'll give it another shot later with a freshly updated build. > I think this is worthwhile to pursue but I agree that it shouldn't block the > intent. > > LGTM3, thanks! Thanks for the LGTMs. By the way, the ChromeStatus page is still gray due to one missing approval. Could you check it and add your approval if you forgot it? > > 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. > > -- > 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/CADsXd2Mq8AJwc6Nap86%2B%3DWXrT9a6cyptKa4Mr4D5KOJHkRPBGQ%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/34842255-FC56-472E-9BF4-6CD2D7784145%40chromium.org.