ViMusic icon indicating copy to clipboard operation
ViMusic copied to clipboard

Remove "loop none" option

Open vfsfitvnm opened this issue 2 years ago • 18 comments

Currently, the playback has 3 loop options:

  1. Loop none: when the queue ends, the playback ends as well
  2. Loop all: when the queue ends, the playback re-starts from the first song in the queue
  3. Loop one: when the playing song ends, it is restarted

However, I think the "loop none" is useless: in which scenario its behavior is desiderable? Personally, I always make sure it's not selected, because I'd rather listen to the same songs than having silence.

Preview:

https://user-images.githubusercontent.com/46219656/188190769-06630fb1-e258-43a2-a0d9-aee8ec1afa6d.mp4

What do you think?

vfsfitvnm avatar Sep 02 '22 15:09 vfsfitvnm

Hmm I personally do not have any use to it as well, but I think maybe some people might sleep while music is playing and so they'd want their queue to eventually end. You can still go with your idea (I like the icon and it gives it something unique relative to other apps) but you might have to consider adding a sleep timer which cuts the music after a certain amount of time.

Obviously someone who has a genuine use case for it is more fit to speak than me but since nobody commented yet I thought I'd give my opinion.

PassionateG1t avatar Sep 03 '22 17:09 PassionateG1t

The sleep timer is already present :smile:

vfsfitvnm avatar Sep 03 '22 17:09 vfsfitvnm

Regarding the icon, this is the alternative:

https://user-images.githubusercontent.com/46219656/188282467-cf62d3e5-2af3-40aa-9b39-25825086128a.mp4

vfsfitvnm avatar Sep 03 '22 17:09 vfsfitvnm

Lol never noticed it since I have no use for it 😄

PassionateG1t avatar Sep 03 '22 17:09 PassionateG1t

Can you add a "Play until end of Playlist" option to the sleep-timer, because I use "loop none" quite often.

Jak2k avatar Sep 05 '22 12:09 Jak2k

@Jak2k In which situations do you use the loop none?

vfsfitvnm avatar Sep 05 '22 12:09 vfsfitvnm

If I hear a playlist and then want to hear sth. other. When I would append the other playlist, Shuffle would shuffle everything, jot just the second playlist.

Jak2k avatar Sep 05 '22 15:09 Jak2k

I don't get how the "loop none" would help here?

vfsfitvnm avatar Sep 05 '22 15:09 vfsfitvnm

With the option I notive when the playlist is heared. Then I can play the next.

Jak2k avatar Sep 05 '22 17:09 Jak2k

You can simply enqueue the playlist you plan to listen next, so you don't have to worry about it at all (even if the "loop none" option was still present) :smile:

vfsfitvnm avatar Sep 05 '22 17:09 vfsfitvnm

But when I click on shuffle, the old and the new playlist will be mixed

Jak2k avatar Sep 05 '22 17:09 Jak2k

@Jak2k In which situations do you use the loop none?

I personally would use (and have used) "loop none" most of the time. Sometimes I just want to hear one song to motivate myself e.g. for starting to clean teeth. With "loop none", playback would stop after the song ends, but I could finish cleaning teeth and meanwhile think about the song. But now the problem is that the song will repeat, so I don't have this short, but nice silent moment afterwards anymore. Also, songs often have different moods at the beginning and at the end. So when I want to hear one only once, I have to stop playback really quickly after it has ended. Otherwise only the first 2 seconds of the repeated song could through me out of the "end mood" which is really annoying.

thornySoap avatar Oct 14 '22 20:10 thornySoap

I agree with @thornySoap, sometimes you just want to listen to a single song/album/playlist. All music players that I know of have 'loop none' set by default, and "I don't use it so neither do you" is probably not the best excuse for removing a feature like that. Likely 45235ac won't take any work to rollback.

nR3D avatar Oct 16 '22 11:10 nR3D

Just now it happened again. Please don't ask why, but I just heard the complete soundtrack of a Star Wars film. The music of the credits ended, I was hanging around in memories. But then suddenly the opening fanfare blew the mood completely away, because I was not fast enough to stop it. That is very annoying

thornySoap avatar Oct 16 '22 16:10 thornySoap

My point was it's very annoying to cycle through three loop options: I proposed to remove the one I thought was used the least, nobody complained so I proceeded. All I care is the ability to set and unset the "loop one" option with only one tap - and I can already tell you the three loop option cycle won't come back.

However, I understand your points. I see two solutions:

  1. Remove the "loop all" option instead (yet it would cause similar complains)
  2. Long press to set the "loop none" option (it would be hard to discover?)

vfsfitvnm avatar Oct 16 '22 17:10 vfsfitvnm

Of course there's a third option, that is adding an option ion in the settings to disable the automatic queue looping. But, to do so, I must understand how much frequently a user changes the loop mode from "loop all" and "loop none". Is that a user preference, or does it depends on the playlist?

vfsfitvnm avatar Oct 16 '22 18:10 vfsfitvnm

  1. Remove the "loop all" option instead (yet it would cause similar complains)
  2. Long press to set the "loop none" option (it would be hard to discover?)
  1. Yeah, as I previously said those three loop options are offered by all music players, so removing one of them will eventually lead to more complains
  2. It could work, maybe a double tap (to disable looping sort to say) might be easier to discover since it's what people are already doing to cycle between the options. But again, the way it was previously is how new users will expect it to work, so be ready to hear that question a lot.

Is that a user preference, or does it depends on the playlist?

Personally, I use 'loop none' 90% of the time (not necessarily depending on the playlist), but I don't like the idea of digging through the settings for that 10%

nR3D avatar Oct 16 '22 18:10 nR3D

Maybe put the loop all/none option here?

thornySoap avatar Oct 17 '22 08:10 thornySoap

That's a good placement (the "loop all/none" involves the whole queue rather than a single song), however I'm not sure about the icon (even though a simple text would be fine as well) and what should happen when the "loop one" option is enabled as well. What loop mode has the highest importance? Does swapping the queue loop mode also reset the single track loop?

vfsfitvnm avatar Oct 17 '22 10:10 vfsfitvnm

I would say loop one has a higher priority than loop all and changing one of them doesn't change the other one, so accidently tapping it wouldn't be annoying

thornySoap avatar Oct 17 '22 14:10 thornySoap

What do you think?

https://user-images.githubusercontent.com/46219656/196216531-ef24aa56-f324-46a4-a3c3-9b9a56a7043d.mp4

vfsfitvnm avatar Oct 17 '22 15:10 vfsfitvnm

@vfsfitvnm Perfect, Happy with this update.

ARBoyGo avatar Oct 17 '22 15:10 ARBoyGo

@vfsfitvnm Will the selection be kept in memory?

nR3D avatar Oct 17 '22 22:10 nR3D

Of course, it will be stored persistenly

vfsfitvnm avatar Oct 18 '22 05:10 vfsfitvnm

I like it really much too. But couldn't "loop none" be missunderstood since a single song would loop if "loop one" is checked? Maybe "don't loop" would be more concise

thornySoap avatar Oct 18 '22 05:10 thornySoap

Hmm, I could either disable that button when "loop one" in enabled (the user has to unset the "loop one" first, but this is a thing he would have done anyway to make the "loop all/none" option take effect) or find another name. "Don't loop" refers to an action rather than a state, so it may be confusing, either "loop disabled"/"loop enabled" or "queue loop off"/"queue loop on" sounds better

vfsfitvnm avatar Oct 18 '22 05:10 vfsfitvnm

Implemented. I opted for the "queue loop off"/"queue loop on" thing as temporary (?) solution

vfsfitvnm avatar Oct 19 '22 13:10 vfsfitvnm

this is the worst idea you ever had

ghost avatar Oct 27 '22 03:10 ghost

this is the worst idea you ever had

You haven't really read the thread, have you?

thornySoap avatar Oct 28 '22 18:10 thornySoap

Implemented. I opted for the "queue loop off"/"queue loop on" thing as temporary (?) solution

Honestly, We are loving it. Like sleep timer is there, Still I don't need it, Cause I don't know when I'll sleep, And whenever I predict my time, And They stop in middle, (Like I keep my phone far from my hand reach, I use headsets.) So Loop none AKA Queue loop off is just great. Maybe it's about personal preferences.

ARBoyGo avatar Oct 28 '22 18:10 ARBoyGo