Skip to content
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

update select all checkbox #36

Closed
wants to merge 44 commits into from
Closed

update select all checkbox #36

wants to merge 44 commits into from

Conversation

edwinheij
Copy link
Contributor

Changed the select all checkbox as it actually did the same compared to when checkbox wasn't checked at all (when exporting). Now when clicked, only the results on the selected page are selected. The selected items are preserved while navigating. Export will only export the selected items (from different pages).

luanfreitasdev and others added 11 commits August 2, 2021 20:10
…this checkbox wasn't checked at all. Now when clicked, only the results on the selected page are selected. The selected items are preserved while navigating. Export will only export the selected items.
@edwinheij
Copy link
Contributor Author

Resolves #37

# Conflicts:
#	resources/views/bootstrap/50/header.blade.php
#	resources/views/tailwind/2/header.blade.php
#	src/PowerGridComponent.php
@edwinheij
Copy link
Contributor Author

Conflicts resolved

@luanfreitasdev
Copy link
Collaborator

luanfreitasdev commented Aug 20, 2021

Hi @edwinheij .

Unfortunately I will have to close the PR so as not to break the part of exporting all selected records in the entire listing.

However I liked the implementation you made and we will work on it soon, or if you can provide a PR to solve this problem we would be very grateful!.

Greetings, thanks for the contribution

@luanfreitasdev luanfreitasdev linked an issue Aug 20, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Export functionality
3 participants