Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove <style scoped> #1226

Merged
merged 1 commit into from
May 11, 2016
Merged

Remove <style scoped> #1226

merged 1 commit into from
May 11, 2016

Conversation

domenic
Copy link
Member

@domenic domenic commented May 10, 2016

This reverts 29cf39d and many follow-up
refinement commits due to lack of implementer interest. Most of the use
cases served by <style scoped> are better served by <style> elements
inside a shadow root (although the features do work differently, in
that shadow DOM enforces isolation from the cascade, whereas <style scoped> does not).

Fixes #552.

This could use some careful review with regard to the changed examples.

@domenic domenic added the removal/deprecation Removing or deprecating a feature label May 10, 2016
text-decoration: underline;
}
&lt;/style>
<pre>
Copy link
Member

Choose a reason for hiding this comment

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

Remove this newline after <pre>? It's inconsistent with what we do elsewhere.

This reverts 29cf39d and many follow-up
refinement commits due to lack of implementer interest. Most of the use
cases served by <style scoped> are better served by <style> elements
inside a shadow root (although the features do work differently, in
that shadow DOM enforces isolation from the cascade, whereas <style
scoped> does not).

Fixes #552.
@domenic
Copy link
Member Author

domenic commented May 11, 2016

Rebased and problems fixed

@annevk annevk merged commit c2a3b2a into master May 11, 2016
@annevk annevk deleted the style-scoped-removal branch May 11, 2016 19:17
@domenic domenic mentioned this pull request Jun 9, 2016
ExplodingCabbage added a commit to ExplodingCabbage/html that referenced this pull request Jul 7, 2016
These were true when the index of elements was introduced, but obsoleted when whatwg#1226 removed the scoped attribute.

Resolves whatwg#1521
domenic pushed a commit that referenced this pull request Jul 7, 2016
These were true when the index of elements was introduced, but obsoleted
when #1226 removed the scoped
attribute.

Fixes #1521.
alice pushed a commit to alice/html that referenced this pull request Jan 8, 2019
These were true when the index of elements was introduced, but obsoleted
when whatwg#1226 removed the scoped
attribute.

Fixes whatwg#1521.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
removal/deprecation Removing or deprecating a feature
Development

Successfully merging this pull request may close these issues.

None yet

2 participants