-
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disable cells while table loading #13961
Disable cells while table loading #13961
Conversation
@@ -96,6 +96,7 @@ | |||
isLoading = false | |||
" | |||
:attributes="\Filament\Support\prepare_inherited_attributes($getExtraInputAttributeBag())" | |||
wire:loading.attr="disabled" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this work reliably with line 78? I believe that was the main challenge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, I'm not sure, but I can check!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah please check if you can :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for reminding me. I completely forgot 😅. I will check as soon as I can!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops I accidentally deleted the repository. However, i have some free time now. I will create a new repo and investigate. Apologies for the delay 😅 @zepfietje
Description
This was discussed in #7294. This PR aims to fix this issue with the exception of ToggleColumn, I tried but I couldnt get it to disable to during table loading. Maybe Zep knows more 👀
Thanks to @atmonshi for helping me!
Visual changes
Functional changes
composer cs
command.