Redraw Missing SVG for org.eclipse.ui.intro Bundle

[BeckerWdf]

I have the feeling I have drawn this SVG already some years ago when I added the @2x.png file. But I couldn't find it.

[BeckerWdf]

@Michael5601: Can you please review?

[Michael5601]

@Michael5601: Can you please review?

The icon looks good to me, thanks for creating it. :)

Only two remark (which both are not that important):
These bottom three paths seem to have no effect and can thus be deleted reducing the file size and rasterization time.

And you could also make sure that the icon is centered correctly (X == 0).

[BeckerWdf]

These bottom three paths seem to have no effect and can thus be deleted reducing the file size and rasterization time

you are right. I will force push a new version.

[BeckerWdf]

And you could also make sure that the icon is centered correctly (X == 0).

done as well

[HeikoKlare]

Looks good, thank you! Should we merge and also integrate in eclipse.platform for the upcoming release?

[BeckerWdf]

Looks good, thank you! Should we merge and also integrate in eclipse.platform for the upcoming release?

From my side this would be ok. Especially because it's the last missing one in that bundle.

But one should have a look at how it looks like inside the running IDE before merging. I made the experience that you cannot simply judge if it looks good simply have looking the SVG standalone. If it looks good in the running IDE then I think it's ok to add it.

[HeikoKlare]

But one should have a look at how it looks like inside the running IDE before merging. I made the experience that you cannot simply judge if it looks good simply have looking the SVG standalone. If it looks good in the running IDE then I think it's ok to add it.

Done and posted results in:

• Add restore welcome page icon as SVG to org.eclipse.ui.intro eclipse.platform#1881