context doesn't work with `shadow: false`

[smeijer]

Forking off from #65; context doesn't work when creating elements without { shadow: true }.

The cause is that the event listener is attached to a slot in shadow mode, and to a Fragment in light DOM mode. Fragments don't handle the events.

I've confirmed that replacing the Fragment with a real element fixes the issue, by applying the change below. I'm opening this issue instead of creating a pull-request, as wrapping elements in another custom element, might be undesired.

Potential fix, replace these lines:

preact-custom-element/src/index.js

Lines 180 to 181 in 0a8ccf7

	const { useFragment, ...rest } = props;
	return h(useFragment ? Fragment : 'slot', { ...rest, ref });

With:

	const { useFragment, ...rest } = props;
	let tag = 'slot';
	if (useFragment) {
		tag = 'preact-context-bridge';
		rest.style = 'display: contents;';
	}
	return h(tag, { ...rest, ref });

[rschristian]

The use of a Fragment there was a silly mistake on my part, was doing some TDD and shipped once the tests were green evidently. Obvs a complete no-op there and the same behavior can be provided much more intelligently.

Not that this helps in the short term, but the use of Fragment can be chalked up to "Ryan was tired when he got that out the door" 😄