Grouped tracks are considered as a single track.

[Vinzzzze]

consider the grouped tracks as one track so the queued tracks cannot interrupt the grouped tracks play :

• add a parameter to the Playlist::next_row method to handle this
• add a method Song::IsOnSameGrouping
• add the attribute next_song_after_queued_ to remember the position where to go back when the queued rest is finish

add a configuration to define the behavior of the queued tracks in front of the grouped tracks :

• add the GROUPED_BEFORE_QUEUE_DEFAULT constexpr that define the default value for this configuration (default value set to 1 to keep the current behavior and ignore the grouped tracks)
• add the update_grouped_before_queue method to set the playlist objects with the value set in the ui
• use left_grouped_song_before_queue_ & init_grouped_song_before_queue_ to determine what to do next when we have queued tracks while reading grouped tracks
• add the ui in the backendsettingspage,in the background management tab.
• add a tool-tip on the created spinbox to explain how this parameter is used.
• translations had been prepared, but only the french one had been done.

correct the previous row to navigate in the playlist :

• update the Playlist::previous_row code
• use only the navigation in the playlist to determine the previous row
• if we are on queued track read, use next_song_after_queued_ attribute to determine the position where to go back (that is to say that queued tracks are ignored in this navigation)

Values for the added parameter :
0 : wait for the end of all the grouped tracks on the same group than the current read track before playing the queued track(s)
1 : ignore the grouping attribute and read the queued track after the end of the current played track
n : wait for the end of the read of the n tracks of the current played track (the current one is the first of the n). If there is less than n grouped tracks to read, wait for the end of all the grouped tracks (same group than the current read track) to start the read of the queued track(s). If the read track is not a grouped track, the queued track(s) will be read just after it.
No maximum had been set.
I will use it this 0 : never interrupt grouped tracks.

[jonaski]

I'm not sure if I want to add this, have to think about it. I doubt many users use shuffle by grouping and if we were to add something like this it makes sense to have the option for album shuffle too, not just grouping.
One thing I see: There is no need to bring the settings logic into SmartPlaylistSearchPreview, it only uses the Playlist model for view, no play logic is used there.

[jonaski]

Another thing, please don't add translations updates into the PR's, it's just unnecessary noise, it's better to update the translation sources with make update_translations separately after the PR's are merged and let Crowin fill in the new tr's for all the languages.

[Vinzzzze]

Hello,
I removed the translation files, then with the «make update_translations» I can regenerate the en_US file, and I have to create a PR with this file alone ?

For the SmartPlaylistSearchPreview, I use default parameter value. As it is a QObject derivated class, I left the Parent parameter at last position.