← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~meths/openlp/testing into lp:openlp

 

On 20 July 2010 11:35, Jon Tibble <meths@xxxxxxxxxxxxxx> wrote:

>
> .  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.



References