When selecting multiple plugins and performing deactivate, keep selected for next bulk action (if any)

[aubreypwd]

When bulk-deactivating multiple plugins I am usually aiming to delete them as well. In a long list it's a pain to re-select them over again to delete them.

[ashfame]

First pass

[SergeyBiryukov]

first_pass.30976.diff​ seems good, but we have to be careful to avoid creating too long URIs, like in #10762 or #29049.

[swissspidy]

30976.diff​ is an updated patch against trunk that makes the current code more readable using functions like sprintf() and checked(). Just like the initial patch, the checkboxes are still checked after plugin activation and deactivation.

Still need to get used to that functionality. Perhaps it's not needed on plugin activation.

[karmatosed]

This feels like a really nice predictive enhancement. From a UX perspective it's a fair assumption to make.

This uses transients to remember selected plugins so the next screen
still retains the selection.

The reason I went with transients here was to avoid the issue where via
current GET method the URL could end up being too long.

This is another method to keep track of selected plugins on the plugin screen like other patches on https://core.trac.wordpress.org/ticket/30976 but subverts the issue of passing the selections in the URL.

Trac ticket: https://core.trac.wordpress.org/ticket/30976

[aubreypwd]

I have another way of maybe doing this (​https://github.com/WordPress/wordpress-develop/pull/1134) that does not have the URL issue mentioned above, but uses transients which may or may not be frowned upon.

Also the only UX issue I see is that when I act on plugins the selection is remembered ONLY for the next pageload, e.g. if you refresh the page your selection will be lost.

[prbot]

​aubreypwd commented on ​PR #1134:

• [ ] I think I still need to do some work to get the selection to persist when updating plugins