Skip to content

[css-cascade-6] Clarify circular dependency (In wording) in finding scoping limits. - #10295

Merged
tabatkins merged 2 commits into
w3c:mainfrom
dshin-moz:patch-1
Apr 28, 2026
Merged

[css-cascade-6] Clarify circular dependency (In wording) in finding scoping limits.#10295
tabatkins merged 2 commits into
w3c:mainfrom
dshin-moz:patch-1

Conversation

@dshin-moz

Copy link
Copy Markdown
Contributor

Finding scoping limits mention in-scope elements, which in turn mentions scoping limits, which in turn etc etc. Just a wording issue, since it can be inferred that the scoping limit portion is ignored at this time.

Still, I think this is helps reduce confusion on first-read.

Finding scoping limits mention in-scope elements, which in turn mentions scoping limits.

Add wording to ignore the scoping limits portion of in scope, which seems implied.
Comment thread css-cascade-6/Overview.bs Outdated
Inlining the relevant dfn instead, and making it non-inclusive.
@tabatkins

Copy link
Copy Markdown
Member

I inlined the desired half of the definition instead.

Looking over it with fresh eyes, it seemed that "in scope" was probably also the wrong definition anyway, since it includes the scoping root, and I don't think we want scoping limits to be able to match the scoping root itself? That would be easy to hit if you're trying to style a nestable component with @scope (.widget) to (.widget), and would produce a scope with a single element in it, which doesn't seem particularly useful. In the rare case you actually do just want one element like that, just using @scope (.foo) { :scope {...}} works just fine, with no need for a limit.

So as part of inlining it, I also changed "inclusive descendant of the scoping root" to just "descendant of the scoping root".

@tabatkins
tabatkins merged commit 246eae5 into w3c:main Apr 28, 2026
1 check passed
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.

2 participants