LGTM2 On Thu, Jun 15, 2023 at 6:10 PM Yoav Weiss <yoavwe...@chromium.org> wrote:
> > > On Thu, Jun 15, 2023 at 6:04 PM Ian Kilpatrick <ikilpatr...@chromium.org> > wrote: > >> >> >> On Thu, Jun 15, 2023 at 2:36 AM Yoav Weiss <yoavwe...@chromium.org> >> wrote: >> >>> Also, thank you so much for working on this and aligning us with other >>> vendors on this important developer feature!! :) >>> >>> On Thu, Jun 15, 2023 at 11:32 AM Yoav Weiss <yoavwe...@chromium.org> >>> wrote: >>> >>>> >>>> >>>> On Thu, Jun 15, 2023 at 7:49 AM 'Ethan Jimenez' via blink-dev < >>>> blink-dev@chromium.org> wrote: >>>> >>>>> Contact emails >>>>> >>>>> etha...@microsoft.com >>>>> >>>>> >>>>> Explainer >>>>> >>>>> None >>>>> >>>> >>>> Well, a short search revealed this article >>>> <https://ishadeed.com/article/learn-css-subgrid/> (by Ahmed Shadeed) >>>> as a great explainer! >>>> >>>> >>>>> >>>>> Specification >>>>> >>>>> https://www.w3.org/TR/css-grid-2/#subgrids >>>>> >>>>> >>>>> Design docs >>>>> >>>>> >>>>> >>>>> >>>>> https://docs.google.com/document/d/1cpsCmzdDlXUKtMxOUKIoJFyLH54mLVhZnam_z0PriO0/edit?usp=sharing >>>>> >>>>> >>>>> Summary >>>>> >>>>> Implements the CSS Grid Layout Module Level 2 specification, which >>>>> introduces the concept of a "subgrid" to nested grid containers. >>>>> >>>>> >>>>> Blink component >>>>> >>>>> Blink>CSS >>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3ECSS> >>>>> >>>>> >>>>> Search tags >>>>> >>>>> subgrid <https://chromestatus.com/features#tags:subgrid>, css-grid-2 >>>>> <https://chromestatus.com/features#tags:css-grid-2>, css >>>>> <https://chromestatus.com/features#tags:css> >>>>> >>>>> >>>>> TAG review >>>>> >>>>> Review Request for CSS Subgrid · Issue #712 · w3ctag/design-reviews >>>>> (github.com) <https://github.com/w3ctag/design-reviews/issues/712> >>>>> >>>>> >>>>> TAG review status >>>>> >>>>> Issues addressed >>>>> >>>>> >>>>> Risks >>>>> >>>>> >>>>> Interoperability and Compatibility >>>>> >>>>> This is a well-defined specification from the W3C, so we would be >>>>> addressing some interoperability gaps instead of causing them. >>>>> >>>>> >>>>> >>>>> *Gecko*: Shipped/Shipping >>>>> >>>>> >>>>> >>>>> *WebKit*: Shipped/Shipping >>>>> >>>>> >>>>> >>>>> *Web developers*: Strongly positive >>>>> >>>>> >>>>> >>>>> *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?* >>>>> >>>>> >>>>> Debuggability >>>>> >>>>> This feature is an extension of the capabilities of the CSS Grid >>>>> Layout Module, no new properties will be introduced, but a new value >>>>> "subgrid" will be introduced and properly rolled into the DevTools >>>>> repository. >>>>> >>>>> >>>>> Will this feature be supported on all six Blink platforms (Windows, >>>>> Mac, Linux, Chrome OS, Android, and Android WebView)? >>>>> >>>>> Yes >>>>> >>>>> >>>>> Is this feature fully tested by web-platform-tests >>>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>>>> ? >>>>> >>>>> Yes >>>>> >>>> >>>> >>>> https://wpt.fyi/results/css/css-grid/subgrid?label=experimental&label=master&aligned >>>> has a few red spots. Are those being tracked somewhere? Any sense of their >>>> impact on compatibility once we say we `@supports` subgrid? >>>> >>> >> wpt.fyi is just a bit slow - all these tests are passing on ToT except >> "line-names-007.html" which is dependent on a CSSWG resolution from one of >> the issues listed below. >> > > That's great to hear!! > > >> >> We've also asked that the additional tests which have been written be >> added to the interop 2023 effort here: >> https://github.com/web-platform-tests/interop/issues/361 >> >> Ian >> >> >>>> >>>>> >>>>> Flag name >>>>> >>>>> LayoutNGSubgrid >>>>> >>>>> >>>>> Requires code in //chrome? >>>>> >>>>> False >>>>> >>>>> >>>>> Tracking bug >>>>> >>>>> https://crbug.com/618969 >>>>> >>>>> >>>>> Estimated milestones >>>>> >>>>> The below estimates may adjust depending on when fragmentation support >>>>> and the fixes for a couple of invalidation bugs are completed. >>>>> >>>>> >>>>> >>>>> Shipping on desktop >>>>> >>>>> 116 >>>>> >>>>> DevTrial on desktop >>>>> >>>>> 114 >>>>> >>>>> >>>>> >>>>> Shipping on Android >>>>> >>>>> 116 >>>>> >>>>> DevTrial on Android >>>>> >>>>> 114 >>>>> >>>>> >>>>> >>>>> Shipping on iOS >>>>> >>>>> 116 >>>>> >>>>> DevTrial on iOS >>>>> >>>>> 114 >>>>> >>>>> >>>>> Anticipated spec changes >>>>> >>>>> The below issues require spec resolution. These ones have interop risk >>>>> depending on the resulting resolutions, but there is currently no interop >>>>> between browsers on these today. As a result, we don’t consider these to >>>>> block shipping, but we do plan to continue to drive forward resolutions: >>>>> >>>>> - [css-grid-1] Application of grid-positioning properties to >>>>> static position of grid children is inconsistent · Issue #7661 · >>>>> w3c/csswg-drafts (github.com) >>>>> <https://github.com/w3c/csswg-drafts/issues/7661> >>>>> - [css-grid-2] Honoring the specified width/height of a subgrid · >>>>> Issue #8966 · w3c/csswg-drafts (github.com) >>>>> <https://github.com/w3c/csswg-drafts/issues/8966> >>>>> >>>>> >>>>> >>>>> The below issues require spec clarification, but there is already >>>>> consensus between implementations and should not block shipping or result >>>>> in any future risk: >>>>> >>>>> - [css-grid-2] Clamp out-of-range subgrid items before or after >>>>> auto placement? · Issue #8494 · w3c/csswg-drafts (github.com) >>>>> <https://github.com/w3c/csswg-drafts/issues/8494> >>>>> - [css-grid-2] How to properly accommodate margin/border/padding >>>>> of a subgrid with no item on the edges · Issue #7465 · w3c/csswg-drafts >>>>> (github.com) <https://github.com/w3c/csswg-drafts/issues/7465> >>>>> >>>>> >>>>> Link to entry on the Chrome Platform Status >>>>> >>>>> https://chromestatus.com/feature/5663795354533888 >>>>> >>>>> >>>>> Links to previous Intent discussions >>>>> >>>>> Intent to prototype: >>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/MN2PR00MB06691936F5BB8575F43A9704BC569%40MN2PR00MB0669.namprd00.prod.outlook.com >>>>> >>>>> >>>>> >>>>> This intent message was generated by Chrome Platform Status >>>>> <https://chromestatus.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/CO2PR00MB02159FA23FB30A441DAC4B1FBC5BA%40CO2PR00MB0215.namprd00.prod.outlook.com >>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CO2PR00MB02159FA23FB30A441DAC4B1FBC5BA%40CO2PR00MB0215.namprd00.prod.outlook.com?utm_medium=email&utm_source=footer> >>>>> . >>>>> >>>> -- 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/CAL5BFfUm7GpHioPc34QZ7XuKSrC%3DutGDUz0GOphL6S8d5e7r-A%40mail.gmail.com.