[Libreoffice] Icon/Image cleanup

[Andrew C. E. Dent]

Hi Joseph
Firstly - thanks for tackling this 'feature'. Great stuff! Really excited about seeing this in a future release.I have a few points to make, but will try to be brief:
1. MaskColorIMHO this is completely redundant, certainly for icons. Before switching to PNGs that can handle proper transparency, this was obviously used with bitmaps. With 95%+ of our icons we wont have a magenta transparency index colour (RGB:255,0,255), yet this is set throughout the code. It would be good to cleanup this useless Masking... However, I think there is still some artwork (not icons) that may use this. See: Basis\share\config\symbol. Of course these few remaining bitmap (BMP) images can be easily converted to PNGs. Happy to help.Also, is there any reason that many of the icons are coded with the wrong file extension (.bmp vs .png)?...
2. BreakageIt seems the tricky cases you are cleaning up might fix some of the breakage that occurs due to HC dependencies. See an old blog entry here:http://oootweak.blogspot.com/2006/02/icon-dependencies.htmlAnd related issues such as: http://qa.openoffice.org/issues/show_bug.cgi?id=61432I'm happy to retest these if you can send me a Dev build (WinXP or Mac). Is there any plan to make some of these patches available to OOo? 
3. TestingAs mentioned above, I'm really happy to play with any builds. To aid testing I have created two drop in replacement icons sets:http://people.bath.ac.uk/ea2aced/OOo/HC-Hacking.zipThese sets substitute all the HC variant icons (sch, etc.) to highlight if we are picking up the wrong images from anywhere.In the 'Normal' galaxy set, the HC icons are replaced with a white 'N' on a RED background; for the 'HiContrast' set we have a yellow 'H' on a BLACK background.It may also be useful to test against an image pack with all HC variants deleted. As per my blog entry, this was useful in finding dependencies and crashes.Here is my list of search tags for easy scripting and removal of HC variants...http://people.bath.ac.uk/ea2aced/OOo/LibO%20HC%20tags.txt
4. MigrationIt has been mentioned that we should start to piece wise remove the HC icons and place them in a new theme (i.e. ooo_custom_images/hicontrast). If this is the case, it might be wise to package the hicontrast theme (internally) as a tar archive (as for Oxygen, Classic, etc). It will save cluttering up the Git repository, since this theme will only every have minimal updates and I also hope in due course we can work on refactoring the Icon Themes, after which it might make sense to store the unarchived files. However, this means every time we move an icon into the theme, the whole archive is updated... Is it worth just creating the hicontrast theme as it stands now, and then piece wise just deleting the HC variants from default_images?...
Let me know if I can be of any assistance. Thanks again.
All the best,Andrew










 		 	   		  
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20101113/17cb1481/attachment.htm>