-
Notifications
You must be signed in to change notification settings - Fork 294
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
Add is-available-to-element-internals #893
Conversation
Co-authored-by: Anne van Kesteren <[email protected]>
Co-authored-by: Anne van Kesteren <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build fails because csswg.org is down (which hosts part of the build pipeline), but I guess we're waiting for the HTML side here anyway.
Ahh ok, thanks. I was wondering what was broken there. Let me know what changes you want to see on the HTML side. I addressed a few comments from rniwa already. |
The build is still failing because of trailing whitespace. I'll try to fixup and get this merged... |
Thanks for the help on all of these PRs! |
…owRoot; r=smaug See whatwg/dom#893. Differential Revision: https://phabricator.services.mozilla.com/D121565
…owRoot; r=smaug See whatwg/dom#893. Differential Revision: https://phabricator.services.mozilla.com/D121565
Add
"is available to element internals"
flag, for use with the accompanying HTML PR which restricts access toElementInternals.shadowRoot
for shadow roots which were pre-existing before a custom element upgrade/construction. See the discussion on Issue 871 for more context.Preview | Diff