fix: toggleDispatch do not dispatch the right event

[MrYamous]

⚡ PowerGrid - Pull Request

• Bug fix
• Enhancement
• New feature
• Breaking change

Description

See #1993 for reporting / Demo for example

After investigation into code, i noticed the function isn't triggering the good event, this pr aims to fix that

Related Issue(s): #1993

Documentation

This PR requires Documentation update?

• Yes
• No
• I have already submitted a Documentation pull request.

[davidtim]

I think another bug related to this one is that during $this->dispatch('pg-toggle-detail-'.$this->tableName.'-'.$rowId, collapsed: null); it won't show spinner. And if add PowerGrid::detail()->collapseOthers() the expend/collapse button won't show spinner also.

[MrYamous]

I think another bug related to this one is that during $this->dispatch('pg-toggle-detail-'.$this->tableName.'-'.$rowId, collapsed: null); it won't show spinner. And if add PowerGrid::detail()->collapseOthers() the expend/collapse button won't show spinner also.

Tbh i'm not sure to understand what is the spinner you're talking about

[davidtim]

I mean the loading indicator

[MrYamous]

I mean the loading indicator

Oh yes, I think this is a CSS animation specific to the collapse icon

[MrYamous]

I took a quick look : it's a css animation specific to the button that is offered by default by the package. If you want to use a specific button, it can take many forms, and I don't know if it's possible to offer an animation that is consistent with that. I think it's better to add your own classes thanks to class method

return [
    Button::make('toggleDetail', 'Click to Toggle Detail')
        ->class('btn btn-primary btn-sm custom-spinner')
        ->toggleDetail($row->id),
];

[luanfreitasdev]

Thank you @MrYamous