openlp-core team mailing list archive

[Jonathan Corwin <j@xxxxxxxxxxxx>]

Oops, replied to the email rather than launchpad... wonder where that email will end up! Lets try again...

>  We could have two lists (probably horizontal tabs on the media "tab") for "General" and 
> "Backing music" if everything needs to be displayed in media manager?

My only concern is that it is already possible to view backing tracks separately in v1.2. If we have two separate lists, both of which can contain audio tracks it might get confusing for the user as to which to use. Also if I add a backing track to a song, and I later want to use it as background music to an image loop, then I would find it inconvenient to have to add it again to the Media tab for your scenario 3.
However your idea of having "Media" and "Backing Track" tabs within Media is a good one.

> I see the point about having videos also providing audio for songs 
> but I think we need to decide which scenarios we want to support.

I only mentioned the videos providing audio as one reason to just use the Media tab files. I don't see it as essential though. Tim has mentioned about Media and Songs not being connected due to separate plugins. This concerns me, we shouldn't be blocked from interacting with other plugins, as long as it is done in a controlled manner. It can be bad enough having two applications that can't interact with each other, without having a single app which can't interact with itself! :)
 
> An implementation point to consider is how we handle deletion of media.  
> If I delete a media item that is being used as a backing track should 
> the item be deleted, only be removed from media but left as a backing track, 
> or nothing removed with a notification to the user or a mix that lets the user decide?

There would need to some message to the user as to what to do. But if we end up keeping the two types separate then it might not be an issue at all.
-- 
https://code.launchpad.net/~meths/openlp/testing/+merge/30369
Your team OpenLP Core is subscribed to branch lp:openlp.