laravel-livewire-tables icon indicating copy to clipboard operation
laravel-livewire-tables copied to clipboard

Bulk Selections Column Doesn't respect set{el}Attributes

Open inmanturbo opened this issue 2 years ago • 3 comments

Discussed in https://github.com/rappasoft/laravel-livewire-tables/discussions/786

Originally posted by inmanturbo May 22, 2022 Bulk selection column ignores setTheadAttributes and setTdAttributes.

Two easy options for fixing are to use conditional and override from values set with above methods, or add setBulkSelectionsTdAttributes and setBulkSelectionsTheadAttributes, possibly setBulkSelectionsCheckboxAttributes?

inmanturbo avatar Jun 10 '22 12:06 inmanturbo

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jul 11 '22 21:07 stale[bot]

Have you find a solution for that issue? im having the same issue with the bulk actions columm...

santiagoavilez avatar Jul 14 '22 01:07 santiagoavilez

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Aug 13 '22 01:08 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Sep 30 '22 05:09 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Nov 06 '22 21:11 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Dec 14 '22 06:12 stale[bot]

I've done a PoC for this, and it's in available in my fork in the development branch. It's a somewhat crude approach, and I've not written the relevant tests for it, so I won't be issuing a pull request for this just yet.

View Commit Here

lrljoe avatar Jan 15 '23 20:01 lrljoe

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Feb 15 '23 01:02 stale[bot]

I've opened a PR for this functionality. Just bear with me as it'll be a few days before I can write the tests and docs.

Please see PR 1058 for details which should be finished soon.

lrljoe avatar Feb 18 '23 07:02 lrljoe

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Mar 20 '23 08:03 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Apr 20 '23 12:04 stale[bot]