colspan for noDataFound case is not calculated properly with custom action

[chknim]

When action's add, edit, delete are all set to false and action's custom is added, empty table does not set colspan property properly (colspan value is one less than it's supposed to be). I have a solution here below.

ng2-smart-table/src/ng2-smart-table/components/tbody/tbody.component.ts

Line 50 in 8f37d37

this.isActionDelete = this.grid.getSetting('actions.delete');

this.isActionDelete = this.grid.getSetting('actions.delete');
+ this.isActionCustom = !!this.grid.getSetting('actions.custom');

ng2-smart-table/src/ng2-smart-table/components/tbody/tbody.component.html

Line 53 in 8f37d37

<td [attr.colspan]="grid.getColumns().length + (isActionAdd || isActionEdit || isActionDelete)">

- <td [attr.colspan]="grid.getColumns().length + (isActionAdd || isActionEdit || isActionDelete)">
+ <td [attr.colspan]="grid.getColumns().length + (isActionAdd || isActionEdit || isActionDelete || isActionCustom)">

[chknim]

PR here. #631

[gabmarini]

+1

[farissi]

The repo seems to be inactive :( how to get this implemented to the project?

[rizz5091660]

same goes here, no data found row is not correctly calculate number of column when applying settings = {selectMode: 'multi'...}

[hanalice]

PR here. #631

@chknim the PR is not pass the check, so it hasn't been merged. Pls have a look.

[chknim]

Thanks for checking. I'm not using this lib anymore so does not really matter to me. My change is so simple so feel free to port into your own PR.

…

On Tue, Nov 10, 2020 at 2:09 AM hanalice ***@***.***> wrote: PR here. #631 <#631> the PR is not pass the check, so it hasn't been merged. Pls have a look. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#629 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AC5SRU6WNEWLPBKN6TCNQSLSPDRL5ANCNFSM4EF5S4GA> .

[mouazAlkhodari]

@chknim
i have made new PR that do the same thing mentioned here
you can find it here #1307
i hope it could be merged asap