Add SVGs to org.eclipse.pde.spy bundles#1730
Add SVGs to org.eclipse.pde.spy bundles#1730HannesWell merged 2 commits intoeclipse-pde:masterfrom Michael5601:org.eclipse.pde.bundle.spy.SVGs
Conversation
|
Has your ECA expired? |
I already saw this and tried to sign in but the site is down and then I thought this is due to the current problems at eclipse foundation: The weird part is that the ECA is valid for other projects like JDT. |
https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/5921 |
|
Short version - nothing works now |
|
Yes, license checking and ECA seem broken. |
This commit adds SVGs for all icons in the bundles `org.eclipse.pde.spy.bundle`, `org.eclipse.pde.spy.context`, `org.eclipse.pde.spy.css` and `org.eclipse.pde.spy.event` except for the following as these are not available as SVG yet: org.eclipse.pde.spy.context/splash.svg (is not used in this bundle) The bundle `org.eclipse.pde.spy.core` uses only one icon `spyicon.png` in the file `fragment.e4xmi` which is also not available as SVG yet. The bundle `org.eclipse.pde.spy.model` uses only one icon `application_lightning.png` which is also not available as SVG yet. Almost all icons in bundle `org.eclipse.pde.spy.preferences` are named like enabled icons but are disabled looking. I ignored this bundle as no similar SVGs are available.
|
This pull request changes some projects for the first time in this development cycle. An additional commit containing all the necessary changes was pushed to the top of this PR's branch. To obtain these changes (for example if you want to push more changes) either fetch from your fork or apply the git patch. Git patchFurther information are available in Common Build Issues - Missing version increments. |
HannesWell
left a comment
There was a problem hiding this comment.
Thank you @Michael5601 for for providing these SVGs they look all good.
Do you plan to contribute the missing SVGs later?
Looking at the E4 Spies menu via Window -> Spies one can really see the difference:

One can also see that the icons in the top left are poorly scaled up, even with SVGs. I assume that happens on the raw image. But I first have to find out where exactly.
Btw. I noticed that the contributed SVG files either contain file-system links or contain base-encoded PNGs. Is this correct and relevant?

This is only the export path that may be used, e.g., by SVG tools to export to (or propose as export path). I agree that it's not that nice to have that path in the SVGs, but it's the case for many of them (also in Platform UI) and since it does no harm I don't see a need to invest effort into changing that. |
HannesWell
left a comment
There was a problem hiding this comment.
Yes, I assumed that that's the case. Thanks.
Then lets add this as it is now and if one is interested to clean that up later. I hope that a simple search and replace/remove would do the job for all SVGs in the repo.
Thanks for the review :) Yes I plan to contribute the missing SVGs. |

This PR adds SVGs for all icons in the bundles
org.eclipse.pde.spy.bundle,org.eclipse.pde.spy.context,org.eclipse.pde.spy.cssandorg.eclipse.pde.spy.eventexcept for the following as these are not available as SVG yet:org.eclipse.pde.spy.context/splash.svg (is not used in this bundle)
The bundle
org.eclipse.pde.spy.coreuses only one iconspyicon.pngin the filefragment.e4xmiwhich is not available as SVG yet.The bundle
org.eclipse.pde.spy.modeluses only one iconapplication_lightning.pngwhich is also not available as SVG yet.Almost all icons in bundle
org.eclipse.pde.spy.preferencesare named like enabled icons but are disabled looking. I ignored this bundle as no similar SVGs are available.See also this PR for more information.