Skip to content

Add SVGs for Eclipse Logo#2058

Merged
merks merged 1 commit intoeclipse-platform:masterfrom
Michael5601:Eclipse-Icon
Jul 20, 2025
Merged

Add SVGs for Eclipse Logo#2058
merks merged 1 commit intoeclipse-platform:masterfrom
Michael5601:Eclipse-Icon

Conversation

@Michael5601
Copy link
Copy Markdown
Contributor

This PR introduces a SVG for the Eclipse Logo in different sizes e.g. eclipse16.svg, eclipse32.svg etc. and changing the references to SVG. This allows better scaling for the window icon in the top left and the icons in the About dialog.

@Michael5601
Copy link
Copy Markdown
Contributor Author

Michael5601 commented Jul 16, 2025

Please see this comment for clarification why the Eclipse logo needs further refinement.

The stripes in the middle of the Eclipse logo as proposed in this draft are not exactly in the middle of the icon (y=5.5px) for 16x16px size.

image

To allign pixel perfect they either must be at y=6.0px which is too low or at y=5.0px which is too high.

A solution to this would be to change the placement of the logo in the grid. Right now the logo is alligned in the middle of the view area. I do not know if there is any restriction for the placement of the logo in the style guidelines.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 16, 2025

Test Results

 1 947 files  ±0   1 947 suites  ±0   1h 38m 56s ⏱️ - 4m 53s
 4 720 tests ±0   4 696 ✅ ±0   24 💤 ±0  0 ❌ ±0 
14 160 runs  ±0  13 993 ✅ ±0  167 💤 ±0  0 ❌ ±0 

Results for commit edb4534. ± Comparison against base commit b0ec300.

♻️ This comment has been updated with latest results.

@BeckerWdf
Copy link
Copy Markdown
Member

A solution to this would be to change the placement of the logo in the grid. Right now the logo is alligned in the middle of the view area. I do not know if there is any restriction for the placement of the logo in the style guidelines.

I think this is a good solution. We have to make compromises in this case and I think this is a good one.

@HeikoKlare
Copy link
Copy Markdown
Contributor

A solution to this would be to change the placement of the logo in the grid. Right now the logo is alligned in the middle of the view area. I do not know if there is any restriction for the placement of the logo in the style guidelines.

I think this is a good solution. We have to make compromises in this case and I think this is a good one.

Agreed. Do you already have a draft of that version? Would be nice to test around with it in Eclipse, as I am just concerned that the usages as a program icon (in the top left of the window on Windows, in the taskbar etc.) may look a bit strange if the icon is not centered, so that's something we should test.

@Michael5601
Copy link
Copy Markdown
Contributor Author

A solution to this would be to change the placement of the logo in the grid. Right now the logo is alligned in the middle of the view area. I do not know if there is any restriction for the placement of the logo in the style guidelines.

I think this is a good solution. We have to make compromises in this case and I think this is a good one.

Agreed. Do you already have a draft of that version? Would be nice to test around with it in Eclipse, as I am just concerned that the usages as a program icon (in the top left of the window on Windows, in the taskbar etc.) may look a bit strange if the icon is not centered, so that's something we should test.

Not yet. I try to provide it today, so you can test tomorrow. :)

@Michael5601
Copy link
Copy Markdown
Contributor Author

I added the non-centered pixel perfect Eclipse Logos to this draft so one can try them out in the IDE.
I made sure the displacement is consistent for all three sizes:

image image image

I also have an alternative suggestion:
The sizes 32x32px and 48x48px can be improved by moving the logo one pixel above, so it looks more centered. This is not possible for the 16x16px variant as the space is limited. Thus we can improve the looks of the sizes 32x32px and 48x48px by sacrificing the consistency with the 16x16px variant.

@BeckerWdf
Copy link
Copy Markdown
Member

Thus we can improve the looks of the sizes 32x32px and 48x48px by sacrificing the consistency with the 16x16px variant.

That's a good idea. The 16x16 one is so small that this sacrifice is even not visible in real life...

@HeikoKlare
Copy link
Copy Markdown
Contributor

I have taken some screenshots at different zooms. It looks really good to me. The only concern (as I expected) is with respect to the task bar: if your are sensitive, you will notice that the icon is off there (compared to the others around). You can see it in the 100% taskbar screenshot: the middle icon is the new one, the ones around are the old ones. The old ones fit slightly better to the icons around. But maybe that's something that is fixed when using different alignment for the 32x32 and 48x48 versions, as the task bar icons are obviously larger. So I am in favor of following that approach:

Thus we can improve the looks of the sizes 32x32px and 48x48px by sacrificing the consistency with the 16x16px variant.

Title bar

100%
image

150%
image

175%
image

Task bar

100%
image

150%
image

175%
image

@Michael5601
Copy link
Copy Markdown
Contributor Author

As suggested I improved the placement of the 32x32px and 48x48px in the current state of this draft.

image image image

I changed this draft to a PR.

@Michael5601 Michael5601 marked this pull request as ready for review July 18, 2025 10:15
This commit introduces a SVG for the Eclipse Logo in different sizes e.g. `eclipse16.svg`, `eclipse32.svg` etc. and changing the references to SVG. This allows better scaling for the window icon in the top left and the icons in the About dialog.
Copy link
Copy Markdown
Contributor

@HeikoKlare HeikoKlare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really great to me now. Let's bring it into M2 to have more people test it. @BeckerWdf do you have any objections on merging this as is?

@BeckerWdf
Copy link
Copy Markdown
Member

This looks really great to me now. Let's bring it into M2 to have more people test it. @BeckerWdf do you have any objections on merging this as is?

Go for it 👍

@merks merks merged commit ea3f147 into eclipse-platform:master Jul 20, 2025
18 checks passed
@Michael5601 Michael5601 deleted the Eclipse-Icon branch July 21, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants