This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 426024 - [Graphics] icons in PNG and SVG format for platform are missing
Summary: [Graphics] icons in PNG and SVG format for platform are missing
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.4   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 4.4 M7   Edit
Assignee: Lars Vogel CLA
QA Contact: Lars Vogel CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 427950
  Show dependency tree
 
Reported: 2014-01-17 15:31 EST by Alfonso Muñoz-Pomer Fuentes CLA
Modified: 2014-04-07 13:44 EDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alfonso Muñoz-Pomer Fuentes CLA 2014-01-17 15:31:29 EST
Several GIF icons in PDE UI are missing their PNG counterparts.

The first twenty or so icons in "icons/obj16" for instance (e.g. "alert_obj.gif", "all_sc_obj.gif", "att_boolean.gif"...).

Additionally there are some folders which are missing completely: "dtool16", "etool16", "eview16" and "wizban").
Comment 1 Lars Vogel CLA 2014-01-17 15:37:58 EST
I think we have also some missing icons for JDT UI.

/icons/full/ovr16/java_ovr.png
/icons/full/obj16/jcu_obj.png
/icons/full/obj16/classf_obj.png
/icons/full/obj16/jsearch_obj.png
/icons/full/obj16/jworkingSet_obj.png
/icons/full/obj16/java_model_obj.png
Comment 2 Lars Vogel CLA 2014-01-17 15:39:18 EST
@Tony, could you please have a look?
Comment 3 Tony McCrary CLA 2014-01-18 08:45:40 EST
(In reply to Lars Vogel from comment #2)
> @Tony, could you please have a look?

I'll check it out. I know there are still few icons that aren't perfectly aligned to pixels causing a blurry effect, specifically in PDE.
Comment 4 Tony McCrary CLA 2014-01-18 08:57:58 EST
(In reply to Tony McCrary from comment #3)
> (In reply to Lars Vogel from comment #2)
> > @Tony, could you please have a look?
> 
> I'll check it out. I know there are still few icons that aren't perfectly
> aligned to pixels causing a blurry effect, specifically in PDE.

Okay, the PDE icons do appear to be missing.

The JDT icons specified do seem to exist on the google code site, so I guess we just missed some during the initial contribution: https://code.google.com/p/eclipse-svg-icons/ It's possible that there's an issue rendering to PNG, the error handling isn't extremely robust. However, I don't recall any errors on the current set hosted by eclipse.org.

I will get working on the PDE icons, sorry about that. If any other JDT icons end up missing let me know.

Also if you notice any icons that need some work (blurry, looks wrong, etc) let me know.
Comment 5 Lars Vogel CLA 2014-01-20 07:23:45 EST
I found a typo in an icon from eclipse.debug.ui Fixed with https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=dce1b1516eab604b02e698e6ac1cfc3ce10bfea6
Comment 6 Lars Vogel CLA 2014-01-21 05:44:57 EST
(In reply to Tony McCrary from comment #4)

> The JDT icons specified do seem to exist on the google code site, so I guess
> we just missed some during the initial contribution:
> https://code.google.com/p/eclipse-svg-icons/ It's possible that there's an
> issue rendering to PNG, the error handling isn't extremely robust. However,
> I don't recall any errors on the current set hosted by eclipse.org.

Looks like the following folders are missing in the Eclipse repo.

/git/eclipse-svg-icons/src/main/resources/eclipse.jdt.ui/org.eclipse.jdt.ui/icons/full/ovr16

I now included them into the Eclipse repo, assuming that these are the most recent versions.

I think wizban icons are already png files in jdt.ui which is probably why you did not create svg versions, correct?
Comment 7 Lars Vogel CLA 2014-01-21 05:54:37 EST
This review adds the missing icons, I have to wait until master is open again to apply it.

https://git.eclipse.org/r/#/c/20862/
Comment 8 Lars Vogel CLA 2014-01-21 06:24:02 EST
(In reply to Lars Vogel from comment #7)
> This review adds the missing icons, I have to wait until master is open
> again to apply it.
> 
> https://git.eclipse.org/r/#/c/20862/

Applied with https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=4d5dee2b51e5f04f33e7b3134bc818d6e977b3c7
Comment 9 Lars Vogel CLA 2014-01-21 06:24:55 EST
More missing JDT icons, this time I really wait until master is open again. ;-)

https://git.eclipse.org/r/20863
Comment 10 Lars Vogel CLA 2014-02-11 18:27:32 EST
(In reply to Tony McCrary from comment #4)

> 
> Okay, the PDE icons do appear to be missing.

> I will get working on the PDE icons, sorry about that. If any other JDT
> icons end up missing let me know.

Any news Tony on the PDE icons?
Comment 11 Tony McCrary CLA 2014-02-12 00:12:10 EST
(In reply to Lars Vogel from comment #10)
> (In reply to Tony McCrary from comment #4)
> 
> > 
> > Okay, the PDE icons do appear to be missing.
> 
> > I will get working on the PDE icons, sorry about that. If any other JDT
> > icons end up missing let me know.
> 
> Any news Tony on the PDE icons?

I'm on the last 10%. I will make an effort to get them ready for the end of the week.
Comment 12 Tony McCrary CLA 2014-02-16 23:28:30 EST
(In reply to Tony McCrary from comment #11)
> (In reply to Lars Vogel from comment #10)
> > (In reply to Tony McCrary from comment #4)
> > 
> > > 
> > > Okay, the PDE icons do appear to be missing.
> > 
> > > I will get working on the PDE icons, sorry about that. If any other JDT
> > > icons end up missing let me know.
> > 
> > Any news Tony on the PDE icons?
> 
> I'm on the last 10%. I will make an effort to get them ready for the end of
> the week.

I'm afraid I'm a bit behind schedule with this. I will update the ticket when the new icons are ready.
Comment 13 Lars Vogel CLA 2014-03-31 17:43:05 EDT
(In reply to Tony McCrary from comment #12)
> I'm afraid I'm a bit behind schedule with this. I will update the ticket
> when the new icons are ready.

Any update Tony?
Comment 14 Lars Vogel CLA 2014-04-07 13:42:13 EDT
(In reply to Lars Vogel from comment #9)
> More missing JDT icons, this time I really wait until master is open again.
> ;-)
> 
> https://git.eclipse.org/r/20863

Looks like I forgot to commit this.
Comment 15 Lars Vogel CLA 2014-04-07 13:43:39 EDT
With https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=e6f14488aed067af2e81279b4885ca5d0fdebf69 I mark this one as fixed. If we find more missing icons I open new bugs.
Comment 16 Lars Vogel CLA 2014-04-07 13:44:06 EDT
.
Comment 17 Lars Vogel CLA 2014-04-07 13:44:36 EDT
Change is visible in Git repo, marking as verified