Cycle in playlists

Nanno Langstraat langstr at gmail.com
Thu Mar 25 18:43:05 CET 2010


Nathan Sala wrote:
> So do you think it would be more logic if this is activated only on 
> "repeat playlist mode" ?

"Repeat playlist" mode already had that behaviour.

By the way, the commit is 1df0724104b2dfb2fbda6d75c2f8c4018a1fad6e

If you want to keep the new "next" behaviour but revert the "back" 
behaviour, use this patch:

@@ -39,3 +39,3 @@ Playlist::StandardTrackNavigator::likelyLastTrack()
-    // Don't make wrap-around conditional on 'm_repeatPlaylist': the user is explicitly asking for this.
     if( lastRow < 0 )
-        lastRow = m_model->qaim()->rowCount() - 1;    // This row is still invalid if 'rowCount() == 0'.
+        if( m_repeatPlaylist )
+            lastRow = m_model->qaim()->rowCount() - 1;    // This row is still invalid if 'rowCount() == 0'.



    Nanno



More information about the Amarok-devel mailing list