Skip to content

feat: CSS nesting baseline support#87

Merged
nzakas merged 1 commit into
eslint:mainfrom
rviscomi:css-nesting
Mar 17, 2025
Merged

feat: CSS nesting baseline support#87
nzakas merged 1 commit into
eslint:mainfrom
rviscomi:css-nesting

Conversation

@rviscomi

Copy link
Copy Markdown
Contributor

Prerequisites checklist

What is the purpose of this pull request?

Add support for evaluating baseline support for CSS nesting

What changes did you make? (Give an overview)

  • added a test for CSS nesting
  • added a handler for the NestingSelector key that checks for nesting selector availability
  • added a check to isSupported so unknown feature statuses are supported by default (if the nesting ID happens to change, the test will fail and there won't be a runtime error)

Related Issues

fixes #85

Is there anything you'd like reviewers to focus on?

Should the nesting selector ID be saved as a constant somewhere, or is it ok to hard-code it in the NestingSelector handler?

@nzakas nzakas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

Should the nesting selector ID be saved as a constant somewhere, or is it ok to hard-code it in the NestingSelector handler?

I think it's fine to keep it local to where it's being used. 👍

@nzakas nzakas merged commit 8fcae6c into eslint:main Mar 17, 2025
@github-project-automation github-project-automation Bot moved this from Needs Triage to Complete in Triage Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

Rule Change: Check CSS nesting syntax for baseline availability

2 participants