#1375 Update Shadow DOM to v1 spec

Open
opened 1 year ago by mattatobin · 3 comments
mattatobin commented 1 year ago (Migrated from github.com)

ShadowDOM which as-implemented isn’t even fully v0 complient needs to be updated to v1 spec so that it will work with Custom Elements v1.

The real problem is that in the course of Mozilla development they hooked stuff into Stylo. This is why I have assigned @wolfbeast to the issue as well. It will require some layout/style changes that will need to be done or evolve along a non-Stylo path.

This will also impact SVG if we choose to include it. Could leave it for now though..


Blocks #1361

ShadowDOM which as-implemented isn't even fully v0 complient needs to be updated to v1 spec so that it will work with Custom Elements v1. The real problem is that in the course of Mozilla development they hooked stuff into Stylo. This is why I have assigned @wolfbeast to the issue as well. It will require some `layout/style` changes that will need to be done or evolve along a non-`Stylo` path. This will also impact SVG if we choose to include it. Could leave it for now though.. ---- Blocks #1361
mattatobin commented 1 year ago (Migrated from github.com)
Owner

An incremental merge will be incoming in the next few days. This will be done bugwise as in one commit per bug collapsing parts in a manor similar to how the mailnews-merge was handled.

After that any work on WebComponents will be blocked by de-unifing all of dom and layout performed by @wolfbeast. Assuming @g4jc has anything more that he is willing to contribute and can be scraped from Mozilla sources will have to be done after the deprot is fixed.

Then we will re-evaluate the plan of action for this and related issues.

An incremental merge will be incoming in the next few days. This will be done bugwise as in one commit per bug collapsing parts in a manor similar to how the `mailnews-merge` was handled. After that any work on `WebComponents` will be blocked by de-unifing all of `dom` and `layout` performed by @wolfbeast. Assuming @g4jc has anything more that he is willing to contribute and can be scraped from Mozilla sources will have to be done after the deprot is fixed. Then we will re-evaluate the plan of action for this and related issues.
wolfbeast commented 1 year ago (Migrated from github.com)
Owner

With /dom/ and /layout/ de-unified (or at least to the point where it shouldn’t cause any further random build trouble), this should be good to work further on, @g4jc

With `/dom/` and `/layout/` de-unified (or at least to the point where it shouldn't cause any further random build trouble), this should be good to work further on, @g4jc
g4jc commented 10 months ago (Migrated from github.com)
Owner

See also: List of bugs gathered from shadow dom meta bugs to sort through. Some include devtools, libedtor, focus navigation, random fixes.

These are still needed for spec:

Shadow DOM v1

See also: [List of bugs gathered from shadow dom meta bugs to sort through](https://dpaste.org/zQAj). Some include devtools, libedtor, focus navigation, random fixes. These are still needed for spec: **Shadow DOM v1** - [x] [Bug 1429656](https://bugzilla.mozilla.org/show_bug.cgi?id=1429656) Implement ShadowRoot.activeElement - [PR 1591](https://github.com/MoonchildProductions/UXP/pull/1591) - [x] [Bug 1430299](https://bugzilla.mozilla.org/show_bug.cgi?id=1430299) Add DocumentOrShadowRoot - [PR 1591](https://github.com/MoonchildProductions/UXP/pull/1591) - [ ] [Bug 1413834](https://bugzilla.mozilla.org/show_bug.cgi?id=1413834) Implement sequential focus navigation regarding shadow DOM - [ ] [Bug 1430303](https://bugzilla.mozilla.org/show_bug.cgi?id=1430303) Implement ShadowRoot.pointerLockElement - [ ] [Bug 1430305](https://bugzilla.mozilla.org/show_bug.cgi?id=1430305) Implement ShadowRoot.fullscreenElement - [ ] [Bug 1413102](https://bugzilla.mozilla.org/show_bug.cgi?id=1413102) Ensure Shadow DOM boundaries are dealt properly in event handling - [ ] [Bug 1438129](https://bugzilla.mozilla.org/show_bug.cgi?id=1438129) Remove ShadowRoot.applyAuthorStyles. - [ ] [Bug 1430301](https://bugzilla.mozilla.org/show_bug.cgi?id=1430301) Implement ShadowRoot.elementFromPoint/elementsFromPoint - [ ] [Bug 1425864](https://bugzilla.mozilla.org/show_bug.cgi?id=1425864) Ensure printing documents which have ShadowDOM works - [ ] [Bug 1066965](https://bugzilla.mozilla.org/show_bug.cgi?id=1066965) make contentEditable and spellchecking to work in ShadowDOM - [ ] [Bug 1326028](https://bugzilla.mozilla.org/show_bug.cgi?id=1326028) customElements.define must upgrade custom element in shadow-including tree order - [ ] [Bug 1451037](https://bugzilla.mozilla.org/show_bug.cgi?id=1451037) check if shadow host has children when fetching before/after pseudos - [ ] [Bug 1452640](https://bugzilla.mozilla.org/show_bug.cgi?id=1452640) Implement the functional :host(..) selector. - **Servo Only** - [ ] [Bug 1453789](https://bugzilla.mozilla.org/show_bug.cgi?id=1453789) Remove Element.createShadowRoot - [ ] [Bug 1454162](https://bugzilla.mozilla.org/show_bug.cgi?id=1454162) Fix cascade order of !important in shadow trees. - [ ] [Bug 1443722](https://bugzilla.mozilla.org/show_bug.cgi?id=1443722) Implement customElements.upgrade - [ ] [Bug 1410578](https://bugzilla.mozilla.org/show_bug.cgi?id=1410578) Make `<link rel="stylesheet">` work in shadow trees. - [ ] [Bug 1331334](https://bugzilla.mozilla.org/show_bug.cgi?id=1331334) Implement :defined pseudo-class for custom elements - [ ] [Bug 1500273](https://bugzilla.mozilla.org/show_bug.cgi?id=1500273) Ensure backward focus navigation works in Shadow DOM. - [ ] [Bug 1595050](https://bugzilla.mozilla.org/show_bug.cgi?id=1595050) Update spec comment in CustomElementRegistry::Define to reflect the latest spec - [ ] [Bug 1552313](https://bugzilla.mozilla.org/show_bug.cgi?id=1552313) Implement custom element disabledFeatures and disableInternals - [x] [Bug 1425685](https://bugzilla.mozilla.org/show_bug.cgi?id=1425685) Implement HTMLSlotElement.assignedElements - 222d316c835019ce64603180e4a73b231b788d76 - [ ] [Bug 1518795](https://bugzilla.mozilla.org/show_bug.cgi?id=1518795) Properly track responsive content in a connected ShadowRoot.
Moonchild modified the milestone from 29.0.0 to 30.0.0 3 months ago
Moonchild added the
Bounty
label 3 months ago
Moonchild referenced this issue from a commit 3 months ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.