search

jeroennoten / Laravel-AdminLTE

Easy AdminLTE integration with Laravel
MIT License
3.78k stars 1.08k forks source link

Fix inputColor wire live #1275

Closed rjpmestre closed 2 months ago

rjpmestre commented 2 months ago
Question Answer
Pull request type ISSUE
License MIT

What's in this PR?

This PR addresses (partially) the issue I described here. Not sure this is the best approach to deal with it. At very best its missing a wrap to check if livewire is active. Something in the lines of @if(config('adminlte.livewire')) { ... }

Checklist

dfsmania commented 2 months ago

Not sure if this change should be done here, maybe you can put that code in your custom Livewire component, without changing anything in the base code. Adding custom support to other tools like Livewire will just complicate the component, an may conflict with other use cases.

rjpmestre commented 2 months ago

I agree with you. Didn't realized this would be out of scope.