Re: [Libreoffice] [Pushed] Remove duplicate placeholder icons

2010-12-10 Thread Joachim
Hi Andrew, Michael, On 12/09/2010 09:05 PM, Joachim Trémouroux wrote: Andrew, Should I do the actual removal of the duplicate icons? I guess these are the "green cross on blue background" images. Regards, Joachim. Here is a patch that removes the placeholder icons in default_images/res. I a

Re: [Libreoffice] [Pushed] Remove duplicate placeholder icons

2010-12-09 Thread Joachim Trémouroux
Andrew, Should I do the actual removal of the duplicate icons? I guess these are the "green cross on blue background" images. Regards, Joachim. 2010/12/9 Michael Meeks > Hi Joachim, > > On Wed, 2010-12-08 at 19:34 +0100, Joachim wrote: > > The attached patch implements in the ImageList class

[Libreoffice] [Pushed] Remove duplicate placeholder icons

2010-12-09 Thread Michael Meeks
Hi Joachim, On Wed, 2010-12-08 at 19:34 +0100, Joachim wrote: > The attached patch implements in the ImageList class a fallback to load > a default icon if no one is found. Wonderful - patch looks lovely I removed the extra braces for single line statements (seems right from a context or

[Libreoffice] [Pushed] Remove duplicate 'placeholder' icons

2010-11-26 Thread Michael Meeks
Hi there, On Fri, 2010-11-26 at 16:01 +0100, Joachim Trémouroux wrote: > Here is another patch for the duplicate icons task. Great :-) > Where ImageList::GetImage is called > from, I don't know, we should audit that too. .. > g grep "vcl/image.hxx" | wc -l returns 179