CappielloAntonio / tempo

An open source and lightweight music client for Subsonic, designed and built natively for Android.
GNU General Public License v3.0
865 stars 39 forks source link

[Feature Request] - Remove song from playlist #174

Open jvadair opened 4 months ago

jvadair commented 4 months ago

Summary

I just downloaded the app today, and realized it answers most of my complaints with Subsonic streaming apps - the big one being the ability to edit playlists. However, I cannot find a way to remove an individual song from a playlist.

Description

I think the easiest way to implement this feature would be to place it under the hold-tap menu for a song in the playlist viewer.

Use Case

Adding this feature would allow users to fully manage their playlists, and, if it is already implemented and I somehow missed it, adding the option to the suggested menu would make the feature much more intuitive.

Additional context

I really love the design and functionality of Tempo, keep up the great work!

CappielloAntonio commented 4 months ago

You can long press playlist item, and edit as you like:

Screenshot_20240219-154238_Tempo

jvadair commented 4 months ago

That makes sense - I think I initially tried swiping right and forgot to try swiping left. Nevertheless, the option is a bit obscure. Would it also be possible to implement into the context/hold-tap menu for an individual song in the playlist? I think I might even be able to implement this, even without knowing the tech stack - only I can't figure out the directory structure :P

Screenshot_2024-02-19-23-35-10-25_8fff24685b220ab505c5c7f79d4b9bbf Screenshot_2024-02-19-23-35-52-37_8fff24685b220ab505c5c7f79d4b9bbf

jvadair commented 4 months ago

I took a look at the code and attempted to modify it in order to add this option to the relevant menu. In doing so, I realized that the same bottom sheet menu is used in other contexts outside of the playlist editor. As an easier solution, I propose adding a delete button next to the drag bar, or on top of the song art (to avoid accidental removal when trying to reorder songs). It might also be possible to detect which context the bottom sheet was called under. I understand why this feature may be impractical, but currently the option is a bit difficult/unintuitive to find without prior knowledge.