- π¨π¦Canada ambient.impact Toronto
Added link to Symfony UX Turbo documentation.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Thank you so much for taking the time to do this write-up, @afinnarn, and for refining it, @Ambient.Impact! π€© π
Hesitation
I am hesitant/concerned about adopting https://turbo.hotwired.dev/ because of the big notice you can see at the top of https://www.drupal.org/project/turbolinks β
They were very aggressive about it.
Then again β¦ https://github.com/symfony/ux-turbo seems to have been around for a while and they somehow did not get shouted at by the creators of Turbolinks? ππ€·
Technically
Technically speaking, I'm not at all opposed to reusing a Symfony component! π
+1 for @Ambient.Impact's proposal to (at least initially) require BigPipe to be disabled.
Drupal.behaviors would be converted to use Stimulus controllers.
This sounds like it would be impossible to make it work automatically for contrib/custom modules'
Drupal.behaviors
? π π± Sounds like @Ambient.Impact has similar concerns.
We'd need to figure out a way to make this work transparently, otherwise this seems not viable. (I did not look into what "Stimulus controllers" are.)Similar concerns apply to the loading of additional CSS/JS.
Conclusion
Big +1 for creating a new branch of Refreshless that adopts this! But prior to building it all, I think we should prototype the foundations to ensure it can work as early as possible, without spending weeks/months on building this. Happy to provide reviews π
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
OMG, it looks like this prototype now already exists!?
- π³π±Netherlands sebastian hagens Breda, Netherlands
@WimLeers
@Ambient.Impact showed me a video of the prototype through Mastodon a while ago, but I can't find it anymore.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
I watched presumably the same video in in a Drupal Slack DM earlier today.
The next step here is to i) review, ii) manually test π Hotwire Turbo minimum viable implementation Active . You could help with that, @Sebastian Hagens π
- π¨π¦Canada ambient.impact Toronto
@Wim Leers
I am hesitant/concerned about adopting https://turbo.hotwired.dev/ because of the big notice you can see at the top of https://www.drupal.org/project/turbolinks β
They were very aggressive about it.
For what it's worth, I read through the exchange you had with them and was appalled at how they treated you. Sure, there's some reasonable discussion they could have had about naming and branding, but they threw that out the window.
Then again β¦ https://github.com/symfony/ux-turbo seems to have been around for a while and they somehow did not get shouted at by the creators of Turbolinks? ππ€·
It's probably because they use the official Turbo that it's technically okay with them. I think as long as we don't use the Turbo name other than to refer to our own integration with the official Turbo library/framework we should be following their requirements, even if we feel they're overly restrictive/stringent.
This sounds like it would be impossible to make it work automatically for contrib/custom modules' Drupal.behaviors? π π± Sounds like @Ambient.Impact has similar concerns.
We'd need to figure out a way to make this work transparently, otherwise this seems not viable. (I did not look into what "Stimulus controllers" are.)Yeah, I can't really see an easy way to do this automagically that won't break some contrib code. It'd be interesting to explore at some point, but it feels like a whole project in itself. Also, controllers in Stimulus are just JavaScript classes that get automatically attached and detached to various elements. There's a bunch of other stuff that I'm glossing over, but that's how I understand them.
OMG, it looks like this prototype now already exists!?
π
@Sebastian Hagens
@Ambient.Impact showed me a video of the prototype through Mastodon a while ago, but I can't find it anymore.
Just uploaded it now to YouTube: https://www.youtube.com/watch?v=KYd7vQZRqMo
- Status changed to Postponed
9 months ago 3:42pm 21 February 2024 - π¨π¦Canada ambient.impact Toronto
See π Hotwire Turbo minimum viable implementation Active for more information.