Proof of concept: Shadow DOM support

[petehunt]

We can already render into shadow roots, this diff lets us listen to events inside shadow DOM.

[euforic]

👍

[zpao]

Can you close #1167 if that's now out of date & not necessary?

[stuarts]

I'd be interested in this pull request being merged, if it's in the cards.

[sebmarkbage]

We'll have a different way of enabling custom elements that doesn't expose the internal class.

This needs to take into account detached containers. Surprised we don't have a test for that.

[stuarts]

Is there a branch for that or is it just in the planning stages?

[mathieuancelin]

Hey guys, any news on that one ?

The changes on ReactDOMComponent.js really makes the event system works within shadow DOM and it would be very cool to be able to mix React components and webcomponents using the shadow DOM.

[futurechan]

👍

[ziahamza]

Any plans of implementing this? There are many use cases where this can be really helpful, expecially for building browser extentions where shadow dom is used to ecapsulate the CSS and the DOM from the rest of the page

[kristianmandrup]

Awesome!!! 👍

[molst]

Hi there,
We use React as the primary component library, but also loads remote (HTML-imported) React components (and components written in other frameworks) that needs to be rendered in shadow DOM's. Hope this get merged soon!

[sebmarkbage]

The right solution is to use event.path to dispatch the event to the inner React component's ID. So I'm going to close this out. We'll still implement event handling the shadow DOM but a different technique.

[Wildhoney]

var doc = container.nodeType === ELEMENT_NODE_TYPE ? container.ownerDocument : container;

I've cursed these 3 lines a thousand times when writing the Maple library for React. In the end I took the approach of sending events to each element in the event.path object which seems to be acceptable, but would love to remove the functionality entirely in favour of a pure React approach that is Shadow DOM aware.

[gurkerl83]

+1 on the Shadow DOM aware suggestion!

[DimitryDushkin]

Why not merge?