Randomize selection in playlist

Hm, I'm not really sure I like having this in the context menu. We had a lot of stuff in the context menu in 0.2.x and it was getting a little too large to navigate easily. Perhaps it could go next to the "Randomize Playlist" option in the tools menu instead, since they are related tools?

Hm, I see. I'm going to disagree - I feel that once I have some selection in the playlist, the context menu should to contain actions that can be performed upon those selected things. For example, see: http://library.gnome.org/devel/hig-book/stable/menus-types.html.en#menu-type-popup

Now, I agree, too many items in a menu is not a good thing - perhaps when it comes to that, see if some sub-menus would be appropriate, or perhaps move it to the main menu. It sort of also worries me that "Tools" is a bit of a generic menu name... the things that are there are already fairly unrelated. (Track properties, which is also part of the context menu, is up there too, but none of the rest of the context menu.)

Fixed with r4119 and r4120. Both "Randomize Playlist" as well as "Randomize Selection" can be found in the context menu of playlists. The latter becomes available if more than 1 track has been selected.

Hm. It seems to me that 'randomize playlist' would belong in the context menu of the playlist tab, not while selecting tracks?

I would basically agree but:

1) The "Randomize Selection" item does not make sense if only 1 track is selected. It is better to always have it have a function and applying that to the full playlist is the most reasonable action.

2) The playlist tab bar can be hidden which would prevent you from accessing the "Randomize Playlist" item. (Before provider menus where introduced, the context menu did actually check if the tab bar was hidden and display the tab menu with the playlist menu as submenu instead. But this is a different issue.)