Bug 426116 - Use png files instead of gif files in org.eclipse.debug.ui and org.eclipse.ui.console
Summary: Use png files instead of gif files in org.eclipse.debug.ui and org.eclipse.ui...
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 4.4   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.4 M7   Edit
Assignee: Michael Rennie CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed, noteworthy
Depends on:
Blocks: 426107 427950
  Show dependency tree
 
Reported: 2014-01-20 07:36 EST by Lars Vogel CLA
Modified: 2014-03-12 05:17 EDT (History)
2 users (show)

See Also:


Attachments
Icons (104.70 KB, image/png)
2014-02-12 19:02 EST, Lars Vogel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2014-01-20 07:36:13 EST
gif files looks really bad on a dark background and we got this great contribution from Tony McCrary who have re-drawn all gif files in SVG and created a Mojo to generate png files from them.

I suggest to use them in org.eclipse.debug.ui.
Comment 1 Lars Vogel CLA 2014-01-20 07:36:42 EST
https://git.eclipse.org/r/20815
Comment 2 Michael Rennie CLA 2014-01-29 14:30:26 EST
Wow, the new png files look really nice. Are there plans to also update org.eclipse.ui.console? It is part of platform debug, so it would be convenient if you updated it as part of this bug.

Also, is there a plan to have someone remove all of the unused gif files (keeping in mind some of the gif files are referenced from the platform doc bundles)?
Comment 3 Lars Vogel CLA 2014-02-11 18:32:45 EST
(In reply to Michael Rennie from comment #2)

> Also, is there a plan to have someone remove all of the unused gif files
> (keeping in mind some of the gif files are referenced from the platform doc
> bundles)?

I think the removal of the gif files will lead to a long discussion as other plug-ins might refer to existing gif files. I plan to start this discussion after the png replacement to avoid that people object against this change.

Console is also planned, I update the review soon.
Comment 4 Lars Vogel CLA 2014-02-12 19:01:42 EST
Gerrit review updated with the icons for the console

https://git.eclipse.org/r/#/c/20815/
Comment 5 Lars Vogel CLA 2014-02-12 19:02:28 EST
Created attachment 239886 [details]
Icons

Looks pretty awesome on a dark theme.
Comment 6 Lars Vogel CLA 2014-03-03 16:36:11 EST
(In reply to Michael Rennie from comment #2)
> Wow, the new png files look really nice. Are there plans to also update
> org.eclipse.ui.console? It is part of platform debug, so it would be
> convenient if you updated it as part of this bug.

Would be nice if you have review https://git.eclipse.org/r/#/c/20815/ 

The change should be complete for debug and console and the new icons look IMHO way better than the gif ones.
Comment 7 Michael Rennie CLA 2014-03-11 10:20:08 EDT
(In reply to Lars Vogel from comment #6)
> (In reply to Michael Rennie from comment #2)
> > Wow, the new png files look really nice. Are there plans to also update
> > org.eclipse.ui.console? It is part of platform debug, so it would be
> > convenient if you updated it as part of this bug.
> 
> Would be nice if you have review https://git.eclipse.org/r/#/c/20815/ 
> 
> The change should be complete for debug and console and the new icons look
> IMHO way better than the gif ones.

The new icons from https://git.eclipse.org/r/#/c/20815/ have been reviewed and submitted. We are still missing new icons for ui.external.tools which is also part of the platform debug repo.
Comment 9 Michael Rennie CLA 2014-03-11 14:07:00 EDT
(In reply to Michael Rennie from comment #8)
> This has been merged:
> 
> http://git.eclipse.org/c/platform/eclipse.platform.debug.git/commit/
> ?id=b0f2acfda47696c9cf1b51ca3c4fdf3eb42bf9e0

Should have mentioned that I opened bug 430127 for the external tools icons so we don't have to keep this one open.
Comment 10 Lars Vogel CLA 2014-03-11 14:55:28 EDT
(In reply to Michael Rennie from comment #8)
> This has been merged:
> 
> http://git.eclipse.org/c/platform/eclipse.platform.debug.git/commit/
> ?id=b0f2acfda47696c9cf1b51ca3c4fdf3eb42bf9e0

Thanks Michael for applying the review, this make the icons on a dark theme look so much better.
Comment 11 Lars Vogel CLA 2014-03-12 05:17:27 EDT
I plan to add this to the N&N M7