LGTM1 I had a chat with Steinar today to answer my questions. Out of the open issues, the important ones to resolve before shipping are: https://github.com/w3c/csswg-drafts/issues/7850 https://github.com/w3c/csswg-drafts/issues/7971 https://github.com/w3c/csswg-drafts/issues/7972
Those don't seem controversial. My LGTM is assuming spec and tests are updated and that we pass those tests before the feature reaches stable. The final test failure is related to #7850 and is an easy fix. Regarding the syntax, there's still disagreement in the CSSWG. Unanimous consensus among all WG members doesn't seem possible here, for any proposal. Crucially, other browser vendors appear to be OK with "option 3". I would definitely reconsider my LGTM if there were signs that other browser vendors are not keen on shipping option 3, since that would create an interop problem, and eventually site compat problems as well. As always, we should be receptive to new information even after an intent has been approved and the flag has been flipped. On Wed, Jan 25, 2023 at 11:20 AM Manuel Rego Casasnovas <r...@igalia.com> wrote: > Adding to Philip questions, there seems to be quite a lot of ongoing > discussions around this topic on the CSSWG, for example today there's a > special meeting only for CSS Nesting topics: > https://lists.w3.org/Archives/Public/www-style/2023Jan/0011.html > > What's their impact on the current implementation? > > Thanks, > Rego > > On 23/01/2023 18:00, Philip Jägenstedt wrote: > > I think that we should ship this. It's a high profile and in-demand new > > feature > > <https://2022.stateofcss.com/en-US/usage/#missing_features_freeform>, so > > I have a few questions and comments first. > > > > Taking a look at the open spec issues > > (https://github.com/w3c/csswg-drafts/labels/css-nesting-1 > > <https://github.com/w3c/csswg-drafts/labels/css-nesting-1>) some are > > explicitly ideas for future changes, but are there any where shipping > > amounts to a decision that isn't easily changed? I'm thinking especially > > of the CSSOM issues. > > > > In https://wpt.fyi/results/css/css-nesting > > <https://wpt.fyi/results/css/css-nesting> there's a single subtest > > failure, related to how a rule serializes. Is that implemented per spec, > > or is it tied up with the open CSSOM issues? > > > > Regarding the threat of a formal objection, there doesn't appear to be > > any solution that would fully satisfy everyone, but I trust your > > judgment that this is the best option. Additionally, we should not > > pre-commit Blink to shipping parser changes, and accept the possibility > > that what we ship now is the final shape of the feature. > > > > On Fri, Jan 20, 2023 at 10:42 AM 'Steinar H. Gunderson' via blink-dev > > <blink-dev@chromium.org <mailto:blink-dev@chromium.org>> wrote: > > > > Contact emails: se...@chromium.org <mailto:se...@chromium.org>, > > futh...@chromium.org <mailto:futh...@chromium.org> > > Explainer: None > > > > Specification: https://drafts.csswg.org/css-nesting > > <https://drafts.csswg.org/css-nesting> > > > > Summary: Add the ability to nest CSS style rules inside other style > > rules, > > combining selectors from the outer with the inner rule for increasing > > modularity and maintainability of style sheets. > > > > Blink component: Blink>CSS > > > > TAG review: https://github.com/w3ctag/design-reviews/issues/791 > > <https://github.com/w3ctag/design-reviews/issues/791> > > > > TAG review status: Pending > > > > Risks: There is a threat of a formal objection in CSSWG. > > > > Interoperability and Compatibility: > > > > Gecko: Positive > > (https://github.com/mozilla/standards-positions/issues/695 > > <https://github.com/mozilla/standards-positions/issues/695>) > > WebKit: Positive > > (https://github.com/WebKit/standards-positions/issues/69 > > <https://github.com/WebKit/standards-positions/issues/69>) > > > > Debuggability > > Nesting style rules will be a big change for editing and displaying > > style rules in the inspector: > > > > - Showing displaying nested rules for matching declarations > > - Editing selectors > > - Inserting nested rules > > - etc... > > > > Tracking issue for devtools support: https://crbug.com/1172985 > > <https://crbug.com/1172985> > > Devtools says they're on track for shipping in M111. > > > > 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? Yes > > > > Flag name: CSSNesting > > > > Requires code in //chrome? False > > > > Tracking bug: https://crbug.com/1095675 <https://crbug.com/1095675> > > > > Estimated milestones > > DevTrial on desktop 109 > > DevTrial on Android 109 > > Shipping 112 > > > > > > Anticipated spec changes: See above. > > > > Link to entry on the Chrome Platform Status: > > https://chromestatus.com/feature/5800613594529792 > > <https://chromestatus.com/feature/5800613594529792> > > > > Links to previous Intent discussions: > > Intent to prototype: > > > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/YzrEmc%2BqlqPv72Au%40google.com > < > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/YzrEmc%2BqlqPv72Au%40google.com > > > > > > -- > > Software Engineer, Google Norway > > > > -- > > 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 > > <mailto:blink-dev%2bunsubscr...@chromium.org>. > > To view this discussion on the web visit > > > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/Y8ph9gk50U2D92f3%40google.com > < > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/Y8ph9gk50U2D92f3%40google.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 > > <mailto:blink-dev+unsubscr...@chromium.org>. > > To view this discussion on the web visit > > > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAARdPYdMTE%3DjWA4AVXeJfGGTZ5WNzQCR4MiHONuZD3gq43PAOg%40mail.gmail.com > < > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAARdPYdMTE%3DjWA4AVXeJfGGTZ5WNzQCR4MiHONuZD3gq43PAOg%40mail.gmail.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/CAARdPYckGUc%3DxjifpXRrOi_UQ2SCXO%2B68GuDAT4r0B%2B8qC4WSw%40mail.gmail.com.