hotwire-django / turbo-django

Unmaintained // An early stage integration of Hotwire Turbo with Django
https://discuss.hotwire.dev/t/django-backend-support-for-hotwire/1570/
Other
385 stars 20 forks source link

Custom turbo-channels-stream-source element can't be self closing #41

Closed TeamTeaTime closed 3 years ago

TeamTeaTime commented 3 years ago

Hello! Thanks everyone for putting this library together.

This isn't major, but I believe that you can't have self closing custom elements (https://developers.google.com/web/fundamentals/web-components/customelements#define) so turbo_stream_source.html should be: <turbo-channels-stream-source signed-channel-name="{{ channel_name }}"></turbo-channels-stream-source> Currently the custom element subsumes the other elements below it, which hasn't broken anything so far for me, but I don't think it's the intended behavior.

scuml commented 3 years ago

Good find - fixed in #44.