openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #21102
Re: [Merge] lp:~tomasgroth/openlp/mupdf into lp:openlp
> 1Q: Does it works on OS X?
Yea, provided that either ghostscript or mupdf is available.
> 2Q: I didnt get the point with PdfViewer class. Dont you use slide's system in
> OpenLP? Do I correctly understand that you just convert PDF to images and then
> show them like images? May be it will be easier to use image plugin for
> showing images after conversion.
I've thought of the same thing, but I have to admit that I'm yet to find a way to bend the system to my need. I'd like the service-item to be a pdf-presentation, but presenting that using the same aproach as the image plugin isn't easy. But maybe I just don't understand the structure of OpenLP yet.
> And 1 more suggestion: may be it is better to use this:
> os.path.isfile(os.path.join(self.get_temp_folder(), u'mainslide001.png'))
> instead of this:
> os.path.isfile(self.get_temp_folder() + u'/mainslide001.png')
I'll change this in the next push.
--
https://code.launchpad.net/~tomasgroth/openlp/mupdf/+merge/174849
Your team OpenLP Core is requested to review the proposed merge of lp:~tomasgroth/openlp/mupdf into lp:openlp.
Follow ups
References