Button's loading icon not showing if `wire:click` has space after the method name

[jtesta66]

Flux version

v2.1.6

Livewire version

v3.6.3

Tailwind version

v4.0.8

Browser and Operating System

Chrome on Windows

What is the problem?

When using wire:click="function (argument...)" and expecting a loading icon on the button, the space before the function arguments prevents the loading icon from rendering

This works:
<flux:button wire:click="dosomething(argument)">Do Something</flux:button>
This doesn't work:
<flux:button wire:click="dosomething (argument)">Do Something</flux:button>

Code snippets

It should be very easy to reproduce with examples I provided. The actual method running in the component doesn't matter.

Component:

function doSomething ($whatever) {
   sleep (5);
}

blade template:
<flux:button wire:click="doSomething ('testing')">Do Something</flux:button>

How do you expect it to work?

I would expect to be able to have a space before the method parameters and still get the loading indicator on the button.

Please confirm (incomplete submissions will not be addressed)

• I have provided easy and step-by-step instructions to reproduce the bug.
• I have provided code samples as text and NOT images.
• I understand my bug report will be closed if I haven't met the criteria above.

[ju5t]

This is not an issue with Flux but it's got to do with Livewire itself.

<button wire:click="foo (100)" wire:loading.remove wire:target="foo (100)">test</button>

Personally, I'm not sure if I'd call it a bug. There isn't a single popular coding style guide that recommends using spaces between the method's name and its properties. You would only use that for control structures, like if, match, for, etc. Even WordPress doesn't, which has the weirdest style guide I know about.

Not my decision though :)

[jtesta66]

Our entire team uses a space before function parentheses. I'm not sure whether that's popular or not is relevant.

It is a bug; the code is/should be equivalent. I can't think of a language that would care whether there's a space there or not. COBOL may.

What you're saying is that the code behind Livewire should care (for some reason? it doesn't elsewhere ...)

Also, yeah, Josh told me this was a Livewire issue, not a Flux issue, but said I could leave it here for now.

[ju5t]

@jtesta66 that's why I said it's my personal opinion, that's not necessarily the same as yours or the maintainers. It's OK that we disagree. I'm not against fixing it by the way.

Also, yeah, Josh told me this was a Livewire issue, not a Flux issue, but said I could leave it here for now.

Ah, it would've saved me (or others) some time to try and help when this was added to the issue. I think it's important information to share.

[jtesta66]

Josh told me this after I posted the issue. But now it's pretty clear it's for Livewire.