Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-30 Thread Jonathan Corwin
Controllers are now completely self contained. Therefore no references to PowerPoint/Impress or u'nt' outside the relevant *controller.py. This will make it easier to just drop in say a keynotecontroller.py should someone buy me a Mac with Keynote installed. I got an email notification as

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp updated

2009-09-30 Thread Jonathan Corwin
The proposal to merge lp:~j-corwin/openlp/presentations into lp:openlp has been updated. Status: Needs review = Superseded -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12625 Your team OpenLP Core is subscribed to branch lp:openlp.

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp updated

2009-09-28 Thread noreply
The proposal to merge lp:~j-corwin/openlp/presentations into lp:openlp has been updated. Status: Needs review = Merged -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12498 Your team OpenLP Core is subscribed to branch lp:openlp.

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-28 Thread Tim Bentley
Review: Approve Simples -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12547 Your team OpenLP Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-28 Thread Jon Tibble
Review: Approve Works for me. -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12547 Your team OpenLP Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-27 Thread Jon Tibble
Review: Approve Looks good -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12472 Your team openlp.org Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp updated

2009-09-27 Thread noreply
The proposal to merge lp:~j-corwin/openlp/presentations into lp:openlp has been updated. Status: Needs review = Merged -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12472 Your team openlp.org Core is subscribed to branch lp:openlp.

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-27 Thread Jonathan Corwin
Jonathan Corwin has proposed merging lp:~j-corwin/openlp/presentations into lp:openlp. Requested reviews: OpenLP Core (openlp-core) Fix output display position/size for powerpoint -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12498 Your team OpenLP Core is

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-27 Thread Tim Bentley
Review: Approve Simples Approved -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12498 Your team OpenLP Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-27 Thread Raoul Snyman
Review: Approve I can see you're a VB programmer, goto :-P -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12498 Your team OpenLP Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-26 Thread Jonathan Corwin
Note my idea for presentationcontrollers is for them to be loaded dynamically like the plugins are. I.e. for PresentationPlugin to have no hardcoded knowledge of which controllers exist, what they are called or which platforms they work on. PresentationTab could be passed the controllers array

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-26 Thread Tim Bentley
Review: Needs Fixing Very good I like it. There is some strangeness on line 281 , 289 and a bit further down. Looks like a class during a merge which will lead to a compile problem. -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12472 Your team openlp.org Core is subscribed

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-25 Thread Raoul Snyman
Review: Approve -- https://code.edge.launchpad.net/~j-corwin/openlp/presentations/+merge/12388 Your team openlp.org Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp updated

2009-09-25 Thread noreply
The proposal to merge lp:~j-corwin/openlp/presentations into lp:openlp has been updated. Status: Needs review = Merged -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12388 Your team openlp.org Core is subscribed to branch lp:openlp.

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-24 Thread Raoul Snyman
Review: Approve We don't have C++ coding standards, so I unfortunately don't have grounds to moan about your coding style :-P -- https://code.edge.launchpad.net/~j-corwin/openlp/presentations/+merge/12310 Your team openlp.org Core is subscribed to branch lp:openlp.

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp updated

2009-09-24 Thread noreply
The proposal to merge lp:~j-corwin/openlp/presentations into lp:openlp has been updated. Status: Needs review = Merged -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12310 Your team openlp.org Core is subscribed to branch lp:openlp.

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-24 Thread Jonathan Corwin
I pushed some more incomplete changes after my original merge request, since I was expecting the merge to only merge the revision I'd requested and not later ones on the same branch. However it seems this is not the case, and my later incomplete changes ended up merged in trunk. This appears

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-24 Thread Jonathan Corwin
Jonathan Corwin has proposed merging lp:~j-corwin/openlp/presentations into lp:openlp. Requested reviews: openlp.org Core (openlp-core) Not entirely sure about the rules for __init__.py, but these changes get rid of the linux errors. --

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-24 Thread Jon Tibble
Review: Approve Looks good. -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12370 Your team openlp.org Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-24 Thread Raoul Snyman
Review: Approve There should be a blank line between the copyright notice and import os -- https://code.edge.launchpad.net/~j-corwin/openlp/presentations/+merge/12370 Your team openlp.org Core is subscribed to branch lp:openlp. ___ Mailing list:

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp updated

2009-09-24 Thread noreply
The proposal to merge lp:~j-corwin/openlp/presentations into lp:openlp has been updated. Status: Needs review = Merged -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12370 Your team openlp.org Core is subscribed to branch lp:openlp.

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-24 Thread Jonathan Corwin
Jonathan Corwin has proposed merging lp:~j-corwin/openlp/presentations into lp:openlp. Requested reviews: openlp.org Core (openlp-core) Got powerpoint viewer working! -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12383 Your team openlp.org Core is subscribed to

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp updated

2009-09-24 Thread Jonathan Corwin
The proposal to merge lp:~j-corwin/openlp/presentations into lp:openlp has been updated. Status: Needs review = Superseded -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12383 Your team openlp.org Core is subscribed to branch lp:openlp.

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-24 Thread Jon Tibble
Review: Approve Looks good. -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12388 Your team openlp.org Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-23 Thread Jonathan Corwin
Jonathan Corwin has proposed merging lp:~j-corwin/openlp/presentations into lp:openlp. Requested reviews: openlp.org Core (openlp-core) Powerpoint viewer changes. Fix support for PPT Viewer 2007, and build release version with free Visual C++ Express 2008 --

Re: [Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-23 Thread Tim Bentley
Review: Approve Totally meaningful! Approved -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12310 Your team openlp.org Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

[Openlp-core] [Merge] lp:~j-corwin/openlp/presentations into lp:openlp

2009-09-17 Thread Jonathan Corwin
Jonathan Corwin has proposed merging lp:~j-corwin/openlp/presentations into lp:openlp. Requested reviews: openlp.org Core (openlp-core) If on Windows, use COM rather than PyUNO for controlling Impress. -- https://code.launchpad.net/~j-corwin/openlp/presentations/+merge/12005 Your team