| Summary: | Java project icon problems | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] JDT | Reporter: | Markus Keller <markus.kell.r> | ||||||||
| Component: | UI | Assignee: | Matthias Becker <ma.becker> | ||||||||
| Status: | VERIFIED FIXED | QA Contact: | |||||||||
| Severity: | normal | ||||||||||
| Priority: | P3 | CC: | daniel_megert, Lars.Vogel, ma.becker, tmccrary | ||||||||
| Version: | 4.7 | ||||||||||
| Target Milestone: | 4.7 M4 | ||||||||||
| Hardware: | PC | ||||||||||
| OS: | Windows 7 | ||||||||||
| See Also: |
https://git.eclipse.org/r/84222 https://git.eclipse.org/r/84224 https://git.eclipse.org/c/platform/eclipse.platform.images.git/commit/?id=98075d54cc8dfdee2878106285acb8d516c1ae95 https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=47ea2556457bd9cb42a672a44da6b72805b6da80 |
||||||||||
| Whiteboard: | |||||||||||
| Attachments: |
|
||||||||||
|
Description
Markus Keller
How can I find out which icon-file this is? Created attachment 265072 [details]
Ganymede Comparison
Are you talking about the java project icon in the explorer view? If so, it's very similar to the original aside from some slight coloration differences.
(In reply to Markus Keller from comment #0) > Created attachment 264671 [details] > Screenshot > > I20161004-1000 > > The Java Project icon doesn't work well with existing overlays. > > The blue flap in the front is not tall enough any more. In the selected > element in the screenshot in the widow on the front, the blue flap is hardly > visible, since it doesn't extend over the bottom rows of pixels that are > typically covered by overlays. > > The same problem probably applies to other project icons as well, so these > should all be fixed in the same way. > > Bug 504629 is tracking problems with the repository overlay. Markus: I tried to find the icon file but wasn't successful. So can you give me a hint were I can find that? (In reply to Matthias Becker from comment #3) > (In reply to Markus Keller from comment #0) > > Created attachment 264671 [details] > > Screenshot > > > > I20161004-1000 > > > > The Java Project icon doesn't work well with existing overlays. > > > > The blue flap in the front is not tall enough any more. In the selected > > element in the screenshot in the widow on the front, the blue flap is hardly > > visible, since it doesn't extend over the bottom rows of pixels that are > > typically covered by overlays. > > > > The same problem probably applies to other project icons as well, so these > > should all be fixed in the same way. > > > > Bug 504629 is tracking problems with the repository overlay. > > Markus: I tried to find the icon file but wasn't successful. So can you give > me a hint were I can find that? There's no Java project icon. What you see is the "normal" project icon: http://git.eclipse.org/c/platform/eclipse.platform.ui.git/plain/bundles/org.eclipse.ui.ide/icons/full/obj16/prj_obj.png decorated with a 'J'. New Gerrit change created: https://git.eclipse.org/r/84222 New Gerrit change created: https://git.eclipse.org/r/84224 (In reply to Dani Megert from comment #4) > There's no Java project icon. What you see is the "normal" project icon: > http://git.eclipse.org/c/platform/eclipse.platform.ui.git/plain/bundles/org. > eclipse.ui.ide/icons/full/obj16/prj_obj.png > decorated with a 'J'. Thank you. That was the missing hint. The offset of the complete graphic inside the icon was wrong. I fixed it. See my two gerrit changes. Created attachment 265117 [details]
Comparison of old (GIF) and new corrected (PNG) version of prj_obj icon
Gerrit change https://git.eclipse.org/r/84224 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.images.git/commit/?id=98075d54cc8dfdee2878106285acb8d516c1ae95 Hey guys, any progress on that. Can we merge the change? Gerrit change https://git.eclipse.org/r/84222 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=47ea2556457bd9cb42a672a44da6b72805b6da80 (In reply to Matthias Becker from comment #10) > Hey guys, > > any progress on that. Can we merge the change? +1 Please reopen if there are still remaining issues. |