Skip to content

Replace invalid test value B&W with B!W since & is valid now with nesting - #39

Open
Jacob Cassidy (jacobcassidy) wants to merge 1 commit into
microsoft:mainfrom
jacobcassidy:update-test
Open

Replace invalid test value B&W with B!W since & is valid now with nesting#39
Jacob Cassidy (jacobcassidy) wants to merge 1 commit into
microsoft:mainfrom
jacobcassidy:update-test

Conversation

@jacobcassidy

Copy link
Copy Markdown
Contributor

Romain Menke (@romainmenke), Here's a small update to the test values to fix it marking the nesting selector & as invalid when appended to a class, such as .main-class { .appended-class& {} } (Ref: CSS Nested Declarations)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this Jacob Cassidy (@jacobcassidy)!
LGTM!

(I don't have write access, so this approval doesn't really mean anything 😅)

@romainmenke

Copy link
Copy Markdown
Contributor

Martin Aeschlimann (@aeschli) Can you take a look at this when you have time?
Thank you 🙇

@aeschli

Copy link
Copy Markdown
Contributor

Thanks Romain Menke (@romainmenke) !

@vs-code-engineering vs-code-engineering Bot added this to the May 2025 milestone Jun 5, 2025
@controversial

Copy link
Copy Markdown

Who has write access here?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants