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

Allow selection of multiple filters (for bulk actions) in Filter editor #4645

Open
contribucious opened this issue Mar 16, 2023 · 3 comments

Comments

@contribucious
Copy link

contribucious commented Mar 16, 2023

Issue Details

Hello! 👋

 
Basically, in the (DNS) Filter Editor, I would like to be able to select multiple filters at once, for bulk actions. Just like with rules.

Why? Especially for list maintainers and frequent private/public rule makers, it would be very useful when doing various tests (to let only User rules enabled quickly, etc.).

Thanks in advance! 👍

 

Proposed solution

🅰️ Basic implementation

Allow to use CTRL and SHIFT keys (combined with mouse — or up/down arrow keys for the latter) to define the selection. And Space bar to toggle the enabled/disabled state on the set.

💡 Advantages
  • Not too complicated to implement I would say and
  • consistent with the fact that already feasible on the rules.
     

🅱️ More advanced implementation (complementary to the first one)

A global checkbox above/below the filters for selection.
Once ticked, a selection checkbox appears on each filter, possibly on the left.

💭 View implementation idea  

In case the left position is chosen, I would say … migrate permanently the state ones to the right, in the form of toggle switches, as used in the AdGuard Browser extension, and make them gray/untouchable during a selection, only left for information about the current status to the user during his selection.

 

🅰️ or 🅱️ Common things …

Reveal …
👁️‍🗨️ In both cases, with an slight adaptation to pluralize in the bottom gear icon menu Enable filter and Disable filter when multiple filters are selected.
👀 In both cases, once multiple filters are selected, ideally show the AdGuard mascot on the right panel — and possibly duplicate there the bottom available actions on them in the form of green buttons: Enable (or Enable selection), Disable (or Disable selection) and maybe also Delete (or Delete selection). Think about mail clients.

 

Alternative solution

Basic implementation only added for now for this v7.

More advanced implementation added later in the v8 — to take advantage of both the renewal of the user interface and the new features¹ scheduled for addition at this place to add a global checkbox at the top or bottom of the list of filters.

¹ See this issue and that one — which is handled in the first one.
@contribucious
Copy link
Author

contribucious commented Mar 16, 2023

For the team (issue handling)

Reveal …

 

Note n°:one:

Regarding the more advanced implementation described above, although I didn't find anything with a search, I'm pretty sure that @gork7777 (or another person) had already proposed this in the past with even a modified screenshot as far as I remember, but probably just in a simple post and not in a dedicated issue apparently.

➡️ Else, just mark my issue as duplicated.

 

Note n°:two:

I guess this issue is a candidate for the filter editor label too.
↪️ As are at least all my other related ones available here (open ones) and there (closed ones). 😉

@Aydinv13
Copy link

@contribucious Hi there!

Definitely a useful improvement, we'll do it. Thanks a lot.

@adguard-bot adguard-bot changed the title (DNS) Filter Editor: Allow selection of multiple filters (for bulk actions) Allow selection of multiple filters (for bulk actions) in Filter editor Mar 28, 2023
@contribucious
Copy link
Author

@Aydinv13 Great! Thank you very much!

P.S. Don't forget my Note n°2 from my second post, just above yours. ☺️
(Except for not using this freshly created label anymore. Or if it's handled in a second time, of course.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants