Make easyMDE and Trix compatible with Livewire 3 #159

alexmanase · 2024-04-03T17:21:49Z

Fixes #80

resources/views/components/editors/trix.blade.php

src/BladeUIKit.php

resources/views/components/editors/trix.blade.php

driesvints · 2024-04-04T12:32:17Z

src/BladeUIKit.php

@@ -57,6 +57,10 @@ public static function outputScripts(bool $force = false): string
            ->sort(fn (string $script) => str($script)->contains('alpine') ? 1 : 0)
            ->map(function (string $script) {
                if (str($script)->contains('alpine')) {
+                    if (config('livewire.inject_assets') && !app()->runningUnitTests()) {
+                        return;
+                    }


Heya, could you explain why you needed to add this?

Because if we have Livewire installed it will inject Alpine JS automatically on every page, so there is not need to include Alpine script if inject_assets is true.

The Dusk tests use Livewire and won't work if Alpine JS in not loaded, so I think in testing environment it's ok to have the Alpine script.

Perfect, thanks @alexmanase

driesvints · 2024-04-04T13:13:42Z

Thank you!

alexmanase added 6 commits April 3, 2024 19:20

do not load Alpine if Livewire has auto_inject

3238e9b

make easyMDE component compatible with Livewire 3

5c72d14

update snapshots

caaf674

always run Alpine in testing environment

97d8072

make trix component compatible with Livewire 3

3cac58f

update snapshots

5e4739c