Skip to content

Conversation

@andruud
Copy link
Member

@andruud andruud commented Oct 21, 2025

<scope-start> is a production, and should use the production markup <<scope-start>>.

Fixes this error:

LINE 1347:22: ''...'' shorthand (opened on 1347:7) was closed,
but there were still open elements inside of it.

`<scope-start>` is a production, and should use the production
markup `<<scope-start>>`.

Fixes this error:

  LINE 1347:22: ''...'' shorthand (opened on 1347:7) was closed,
  but there were still open elements inside of it.
@andruud andruud requested a review from mirisuzanne October 21, 2025 11:46
@mirisuzanne
Copy link
Contributor

That change looks right to me. (I'm not an editor of CSS Nesting)

@andruud
Copy link
Member Author

andruud commented Oct 21, 2025

Ah, I thought you were. My mistake.

@andruud andruud requested a review from tabatkins October 21, 2025 20:34
@SebastianZ SebastianZ merged commit 43aeb2d into w3c:main Oct 22, 2025
1 check passed
@SebastianZ
Copy link
Contributor

That change looks right to me. (I'm not an editor of CSS Nesting)

Neither am I, though I was told by @tabatkins that obvious bugfixes don't require review. And this change definitely falls into that category.

Sebastian

@tabatkins
Copy link
Member

Yes, please just merge obvious bugfixes, no need for review. If you're not sure it's an obvious bugfix, then you can ping for review.

@andruud
Copy link
Member Author

andruud commented Oct 22, 2025

just merge obvious bugfixes, no need for review

In case that was directed at me: I don't have write access to this repo. So I do need a review, effectively.

@tabatkins
Copy link
Member

Ah, fair. Well, you can just ask for a merge, then, so people feel ok just pressing the button. ^_^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants