airdcpp-web / airdcpp-webclient

Communal peer-to-peer file sharing application for file servers/NAS devices
https://airdcpp-web.github.io
171 stars 31 forks source link

Feature Request: Add Remove all bundle and Remove all selected bundle in Action Menu of queue list #416

Closed jaloji closed 2 years ago

jaloji commented 2 years ago

Is your feature request related to a problem? Please describe. I really appreciate the use of the "Remove all completed bundles" button in the Queue->Actions buttons. However, when using the airdcpp-auto-downloader extension it happens that false positives are queued (empty folder in particular), it is very tedious to have to click on each one to remove them.

Describe the solution you'd like It might be interesting to have something like "Remove all completed bundles" but for all bundles "Remove all bundles" or else a checkbox system on the side of each of them and a "Remove all selected bundle" button.

Thank you

peps1 commented 2 years ago

I created an issue for the extension, queuing of empty folders should of course not be happening. https://github.com/peps1/airdcpp-auto-downloader/issues/18

@maksis mentioned before, a checkbox system is not possible in the current implementation and framework used for the table views in the UI

Also on the roadmap for my extension i have to keep track of any items that were queued from the extension and offer a way to remove all items that were queued by it.

jaloji commented 2 years ago

@peps1 Thanks for your answer, but in this case even something like "Remove all bundle" will do the job.

maksis commented 2 years ago

It shouldn't be possible to queue empty folders (or 0 byte files) so I assume that the folders actually contain something. @jaloji I'd recommend going through the problem at https://github.com/peps1/airdcpp-auto-downloader/issues/18 and thinking whether queueing of such folders could somehow be avoided... I'd prefer the feature request https://github.com/airdcpp-web/airdcpp-webclient/issues/90 as multiselect is a more generic solution for removing multiple bundles.

jaloji commented 2 years ago

@maksis Thank you for your answer, indeed request #90 corresponds to my request.

And indeed the files have been queued but are not empty they only contain a .txt file of a few bytes.

maksis commented 2 years ago

Closing as a duplicate of https://github.com/airdcpp-web/airdcpp-webclient/issues/90