dpacassi / disable-javascript

Adds the ability to disable JavaScript on specific sites.
MIT License
275 stars 35 forks source link

[Temporary(?) Feature Request]: Mark and remove multiple whitelisted entries at once #51

Open fastcatch opened 5 years ago

fastcatch commented 5 years ago

Until we have temporary whitelisting I keep my whitelist clean by periodically manually scanning it and removing (usually several) items in each go. However, beyond a near-trivial number of legitimately whitelisted entries it is rather cumbersome to remove many entries: click an entry, scroll to "Remove" (the one entry), scroll to the next to remove, mark, repeat.

Thus it would help quite a lot if multiple entries could be selected (with checkboxes, perhaps) and then have a way to remove them all at once. In fact it could even replace the "Remove All" functionality (if there was an easy way to mark all entries). (If it's too complex, sorting by time added would also help a bit, such that "new" entries can be moved down, near the remove button. It's not perfect but I'd believe is a simpler change.)

dpacassi commented 5 years ago

That sounds indeed like a nice feature and wouldn't take too long to implement. I also never experienced the add-on with a huge number of domains, so pagination would be a topic too.

I am quite busy right now but am open to accept pull requests implementing this. Alternatively maybe someone could sponsor this feature. I'm currently working on getting the web extension released in Opera, afterwards I wanted to evaluate if Patreon could be an option for future features. Will keep you updated on this!

Anyhow, thanks for your input @fastcatch!

jroldroyd commented 5 years ago

I would suggest two changes to the whitelist display:

And, not temporary... make this permanent! Thanks.