nextplayer icon indicating copy to clipboard operation
nextplayer copied to clipboard
verified publisher icon anilbeesetti

Delete multiple videos at once

Open Yashraj254 opened this issue 2 years ago • 5 comments

Fix: #688 featured added: select multiple videos for deletion

https://github.com/anilbeesetti/nextplayer/assets/52420957/b55f1ca9-4a50-487e-90de-9ae0f7fe634f

Yashraj254 avatar Nov 20 '23 06:11 Yashraj254

@Yashraj254, thank you for the pull request, and I'm sorry for not getting back to you sooner. Your work looks really good. I have a small suggestion regarding the user interface: instead of using square checkboxes, could you use circular ones? Also, for items that are not selected, we don't need any checkbox at all. Additionally, I think it would improve the appearance if we added a highlighted background for selected videos to make them more noticeable.

anilbeesetti avatar Feb 13 '24 04:02 anilbeesetti

It's alright, I will implement those changes. In the meantime, could you please review my other PRs?

Yashraj254 avatar Feb 13 '24 08:02 Yashraj254

Does this look better or should i use a different color, any suggestions?

https://github.com/anilbeesetti/nextplayer/assets/52420957/9f588763-0ae6-4892-bf5a-4b80c09bcf0f

Yashraj254 avatar Feb 13 '24 10:02 Yashraj254

This is a very useful feature, but its not available yet in the main build?

Aswin8 avatar May 05 '24 14:05 Aswin8

@anilbeesetti any update on getting this merged with the latest changes?

Pentaphon avatar Aug 24 '24 07:08 Pentaphon