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

[css-shadow-parts] rephrase to make it clear which shadow root #2467

Closed
wants to merge 10 commits into from

Conversation

fergald
Copy link
Contributor

@fergald fergald commented Mar 23, 2018

When dealing with mappings on host elements there are 2 possible meanings to "shadow root" - the one attached to the host and the one containing the host. In the spec, only one makes sense, so it's unambiguous but this change makes it explicit.

@fergald
Copy link
Contributor Author

fergald commented Mar 23, 2018

@tabatkins

@fergald
Copy link
Contributor Author

fergald commented Apr 27, 2018

@tabatkins

@fergald
Copy link
Contributor Author

fergald commented May 7, 2018

pr-preview of this is here it's against my own repo since pr-preview doesn't currently work for w3c/csswg-drafts

@fergald fergald changed the title rephrase to make it clear which shadow root [css-shadow-parts] rephrase to make it clear which shadow root May 7, 2018
Copy link
Contributor

@kojiishi kojiishi left a comment

Choose a reason for hiding this comment

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

Can you also explain why we're adding files to docs/rendered directory? I'm not familiar with it.

@@ -7,7 +7,7 @@ Status: ED
Work Status: exploring
URL: http://drafts.csswg.org/css-shadow-parts/
Editor: Tab Atkins-Bittner, Google, http://xanthir.com/contact/, w3cid 42199
Abstract: This specification defines the ''::part()'' and ''::theme()'' pseudo-elements on <a>shadow hosts</a>, allowing <a>shadow hosts</a> to selectively expose chosen elements from their <a>shadow tree</a> to the outside page for styling purposes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you explain why we want to take these links out?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That shouldn't be happening. I think I took the wrong base for this PR. I will fix that.

@fergald fergald closed this in 82d7b52 Jun 5, 2018
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.

None yet

3 participants