Re: [rox-users] MusicBox 016 Released

[<ke...@ya...>]

 --- Ken Hayber <kh...@so...> wrote: 
> JoeHill wrote:

> OK, seems I missed that one.  It works in the Main
> window by using the Shift key, the cursor changes to
> show a + sign.  I need to add this support to the
> load_args() rpc call.

I suggest making append the default on the playlist
(over-ridden by ctrl) and replace the default on the
main window (overridden by shift, as current), this
being the conventional behavior (well, except for the
overridings, but they certainly shouldn't both be the
same; this way someone can if nothing else work out
that if they use ctrl, it'll *always* append and shift
will *always* replace).

>  I also think now that the
> load call should not automatically start playing. 
> If you want that you can just call play() (you being
> me in this case). 

For the backend, this is appropriate, but I think
dropping a song on the main window should make it play
immediately.

> I should probably also check for
> duplicates and stuff like that.

No. Given the structure and purpose of a playlist,
it's entirely reasonable to have the same song in a
list twice  (hell, depending on my mood I've been
known to have certain songs in playlists practically
every second one). The absolute most you could do
there is pop up an annoying dialog box, but I think
I've given my opinion there... :)

BTW... Is this on 0Install? That'd be a convincing
reason for me to use it :) (ATM I'm happyish with
Rhythmbox, but not entirely happy.)

--
Tristan

Find local movie times and trailers on Yahoo! Movies.
http://au.movies.yahoo.com