amarok Context Menus

Seb Ruiz had a question about whether unusable features should be removed from context menus or grayed out.

In his case he is inquiring about the ability to burn CDs from a playlist. It turns out that the feature he’s referring to was actually originally authored by myself as part of JuK. I’m not sure how it’s changed in amaroK since then, but in JuK the feature was always compiled in. It was only disabled if the CD-burning program was not found installed when JuK started. So it would make sense to leave the option there in the menu but disabled.

Turns out that the option is removed if it can’t be run though, in JuK’s case. Or to be more specific, the option is not added if it can’t be run, since the menu in question is constructed dynamically. Maybe I’ll change that someday since I actually like leaving the options there when possible and just disabling it if it can’t be run, if only to keep the menu from changing around.