[Libreoffice] [Pushed] Remove duplicate placeholder icons

Michael Meeks michael.meeks at novell.com
Thu Dec 9 11:20:29 PST 2010


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 two), otherwise wonderful
stuff. Should help shrink our install size yet further.

> I have performed the following tests:
..
> 2nd test is removing from images.zip all the icons except the default one...
> Find attached a screenshot of the result.

	Really great testing too. Ace - I guess the way is cleared for removing
all the missing icon duplicates (if they are still there). Do you think
you could move the task from the Easy Hacks page into the 'completed
easy hacks' page to avoid others spending time on it :-)

	Good stuff ! what is your next mission ?

	All the best,

		Michael.

-- 
 michael.meeks at novell.com  <><, Pseudo Engineer, itinerant idiot




More information about the LibreOffice mailing list