← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~crichter/openlp/media_rewrite into lp:openlp

 

Thanks for review!

> 855 needs to be a to of class
Im not understand this comment

> Line 2407 - how do I get this to work?
This is "only" a temporarily controller, which will used at loading of media file internally. (no really user interaction possible/needed, but needed to satisfy the interfaces)

> Preview summary in slide controller is in the wrong place.
What do you mean?

The other issues I will fix (hopefully this evening)
There is another issue I found by myself. Playing video with webkit in Preview is only visible after display setup :-(
-- 
https://code.launchpad.net/~crichter/openlp/media_rewrite/+merge/82048
Your team OpenLP Core is subscribed to branch lp:openlp.


References