Look into using `x-ignore` instead of attribute renaming

Accudio / async-alpine

Async Alpine brings advanced code splitting and lazy-loading to Alpine.js components!

https://async-alpine.dev

Apache License 2.0

152 stars 12 forks source link

Look into using `x-ignore` instead of attribute renaming #18

Closed Accudio closed 2 years ago

Accudio commented 2 years ago

This could massively simplify how we handle disabling/enabling Alpine, depending on how it handles x-ignore.

It could also unlock #11, if Alpine checks the live node for x-ignore rather than relying on the NodeList passed with the MutationEvent.

Accudio commented 2 years ago

Experimented a little with this and a few things are of note:

Works well for simple use cases. Adding x-ignore to an x-data will stop all children from running.
As it stops all children, we would not be able to load a child before all parents
Doesn't help with #11, Alpine uses the static NodeList which prevents us from intercepting mutations.

Accudio commented 2 years ago

This would be used with the reengineering as proposed in #11. The breaking of parent loading strategy is unfortunate but makes Async Alpine much easier to manage, maintain, and the dynamic content issue is a priority.

Accudio commented 2 years ago

This has been implemented in the beta 0.4 — #20