Support and General Use > Audio Playback, Database and Playlists
"Add To Playlist" patch
lenny:
thanks buddy
Shebb:
Sorry about the case thing. Silly of me. Since all my playlists start with a capital I forgot the need for a case insensitive compare. New file should correct all that.
I had to take the browse playlists function out of the context menu because it has nothing to do with the file selected. As Linus said, create directory should also be moved from here. If the playlist thing has enough appeal then a button could be assigned to it.
[attachment deleted by admin, too old]
lenny:
thanks shebb. I'll try it later. I wouldn't have noticed if I didn't have one playlist with a lowercase letter.
I don't quite understand why it had to come out of the context menu, but I'll take your word for it. the record button isn't doing anything... :p
but that gets me thinking that there could be a variety of functions for the record button and the user could select which one to asign it from the settings. makes sense to me anyway.
Assimalyst:
Great work Shebb, thanks a lot!
I agree with point 2 that lenny made, think it would be useful.
--- Quote from: lenny on January 21, 2006, 02:34:44 PM ---2) I wish you hadn't taken out the ability to browse the playlists from the context menu. having to go into the main menu is a pain.
--- End quote ---
Also, an additional feature i think would be useful (though i've no idea how difficult it would be to implement), would be a quick scan of the playlist you're adding to, and only adding the file if it's path does not already exist in the playlist, so preventing duplicates. My memory is shocking and i can never remember what i've already added!
What do you think?
Thanks again for an excellent patch.
Sweet Spot:
Linus, what do you think the likelyhood is, of having this patch committed ? I think it's a real step up from the standard implamentation, and would make a fantastic edition to the code.
Doug
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version