Re: [PUSHED] fdo#42070-Fix RTL support in presenter console

2012-11-21 Thread Stephan Bergmann
On 11/20/2012 08:27 PM, Andras Timar (via Code Review) wrote: Thank you for your patch! :-) It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/1053 Approvals: Lior Kaplan: Verified Andras Timar: Verified; Looks good to

Re: [PUSHED] fdo#42070-Fix RTL support in presenter console

2012-11-21 Thread Miklos Vajna
On Wed, Nov 21, 2012 at 08:43:07AM +0100, Stephan Bergmann sberg...@redhat.com wrote: As discussed on gerrit, this does not work and breaks the Mac OS X build. I'll see to get the break fixed by changing the presenter console into a non-extension now. Something realted: doing a git diff on

Re: [PUSHED] fdo#42070-Fix RTL support in presenter console

2012-11-21 Thread Andras Timar
On Wed, Nov 21, 2012 at 9:42 AM, Miklos Vajna vmik...@suse.cz wrote: On Wed, Nov 21, 2012 at 08:43:07AM +0100, Stephan Bergmann sberg...@redhat.com wrote: As discussed on gerrit, this does not work and breaks the Mac OS X build. I'll see to get the break fixed by changing the presenter

[PUSHED] fdo#42070-Fix RTL support in presenter console

2012-11-20 Thread Andras Timar (via Code Review)
Hi, Thank you for your patch! :-) It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/1053 Approvals: Lior Kaplan: Verified Andras Timar: Verified; Looks good to me, approved -- To view, visit