Re: [Libreoffice] [PATCH] impress: after adding a new animation, scroll down to it in the list.

2011-06-26 Thread Luke Symes
Hi Radek, On Sat, Jun 25, 2011 at 3:35 AM, Radek Doulik wrote: > > > At least I am not reaching the part you modified when adding custom > animation thru custom animation pane (using the add button and custom > animation dialog). If we move it to the loop, it will be reached always > when selecti

Re: [Libreoffice] [PATCH] impress: after adding a new animation, scroll down to it in the list.

2011-06-24 Thread Radek Doulik
Hi Luke, thanks for your patch! On Fri, 2011-06-24 at 17:38 +1200, Luke Symes wrote: > Hi there, > > > This is a one-line patch to impress to make the animation list scroll > down to show a newly added animation. Previously we didn't scroll the > list at all. I think better way of doing it mig

[Libreoffice] [PATCH] impress: after adding a new animation, scroll down to it in the list.

2011-06-23 Thread Luke Symes
Hi there, This is a one-line patch to impress to make the animation list scroll down to show a newly added animation. Previously we didn't scroll the list at all. Kind Regards, Luke. From 011c3c21a7a8a250261cead697a4cf5d005a02dd Mon Sep 17 00:00:00 2001 From: Luke Symes Date: Fri, 24 Jun 2011 1

Re: [Libreoffice] [PATCH] impress - remove doubled copyright notice

2011-01-21 Thread Thomas Arnhold
On 01/21/2011 03:08 PM, Michael Meeks wrote: > Hi Thomas, > > On Fri, 2011-01-21 at 14:54 +0100, Thomas Arnhold wrote: >> won't do this without your approval. > > Looks fine to me - no point in duplicating license headers. > > Michael K was writing a nice lint tool, that may be able

Re: [Libreoffice] [PATCH] impress - remove doubled copyright notice

2011-01-21 Thread Michael Meeks
Hi Thomas, On Fri, 2011-01-21 at 14:54 +0100, Thomas Arnhold wrote: > won't do this without your approval. Looks fine to me - no point in duplicating license headers. Michael K was writing a nice lint tool, that may be able to find if this problem (duplicated license headers) aff

[Libreoffice] [PATCH] impress - remove doubled copyright notice

2011-01-21 Thread Thomas Arnhold
Hi folks, won't do this without your approval. -Thomas >From cb55cc9090a98014631949d769263e09152cfa5c Mon Sep 17 00:00:00 2001 From: Thomas Arnhold Date: Fri, 21 Jan 2011 14:52:44 +0100 Subject: [PATCH] Remove old copyright notice. All other files in this dir look the same. So I assume this as

Re: [Libreoffice] [PATCH] Impress

2010-11-23 Thread Sebastian Spaeth
On Tue, 23 Nov 2010 15:33:08 +, Michael Meeks wrote: > Oh - seems reasonable; there is no saving in git repos size by removing > it I suppose. So - you need to checkout: Thanks, I think I'll cope once I find time. > Which is wonderful; that is in now ? perhaps we should back-port

Re: [Libreoffice] [PATCH] Impress

2010-11-23 Thread Michael Meeks
On Tue, 2010-11-23 at 15:57 +0100, Sebastian Spaeth wrote: > By the way, in agreement with ace_dent, I will remove the classic icon > theme that we ship by default with LO. He said that he maintained it and > is ok with not shipping it. Sounds good to me; it is indeed extremely 'classic'

Re: [Libreoffice] [PATCH] Impress

2010-11-23 Thread Sebastian Spaeth
On Mon, 22 Nov 2010 20:55:05 +, Michael Meeks wrote: > PS. really pleased to see the image size decrease in the new theme from > 4.8Mb to 3.3Mb - still lots of low hanging fruit there, with the > duplicate "missing icon" icons, and the res/commandimagelist -> cmd/ > rename to save that space .

Re: [Libreoffice] [PATCH] Impress

2010-11-22 Thread Michael Meeks
Hi there, On Mon, 2010-11-22 at 07:11 -0800, Joseph Powers wrote: > Please disregard my Friday patch for Impress. :-) > Removing the Global High Contrast flag is going to require more work > then the simple remove HC icons project is going to allow. We hard > code too many UI items (text

Re: [Libreoffice] [PATCH] Impress

2010-11-22 Thread Wols Lists
On 22/11/10 15:11, Joseph Powers wrote: > Please disregard my Friday patch for Impress. > > Removing the Global High Contrast flag is going to require more work then the > simple remove HC icons project is going to allow. We hard code too many UI > items (text fonts, background color, border colo

[Libreoffice] [PATCH] Impress

2010-11-22 Thread Joseph Powers
Please disregard my Friday patch for Impress. Removing the Global High Contrast flag is going to require more work then the simple remove HC icons project is going to allow. We hard code too many UI items (text fonts, background color, border color, ...) which have to be modified when in HC mod