Skip to content

Correct SVG icon for showing menu in part stacks and shared images #2995

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

HeikoKlare
Copy link
Contributor

@HeikoKlare HeikoKlare commented May 19, 2025

The SVG version of the icon for showing the menu in part stacks erroneously contained the "thin" version of that icon, showing an error instead of three dots. In addition, the shared image version of that icon contained the three dots shifted to the bottom right and smaller than intended.

This change replaces both SVG version with the same icon containing the correct three-dot-version of the view_menu icon.

Before

image

After

image

The SVG version of the icon for showing the menu in part stacks
erroneously contained the "thin" version of that icon, showing an error
instead of three dots. In addition, the shared image version of that
icon contained the three dots shifted to the bottom right and smaller
than intended.

This change replaces both SVG version with the same icon containing the
correct three-dot-version of the view_menu icon.
@HeikoKlare HeikoKlare marked this pull request as ready for review May 19, 2025 09:36
@HeikoKlare HeikoKlare mentioned this pull request May 19, 2025
@sratz
Copy link
Member

sratz commented May 19, 2025

I don't see this particular version of view_menu.svg (3656 bytes) anywhere else.
Was this corrected for this PR?

@HeikoKlare
Copy link
Contributor Author

Was this corrected for this PR?

Yes, I took the original view_menu.svg from org.eclipse.ui and adapted it. I did not find an SVG version properly representing the existing PNGs, as all versions of that SVG had the three circles shifted to the bottom right and wrongly sized. Maybe I missed the correct one...

@HeikoKlare
Copy link
Contributor Author

Note: I am not an expert in creating SVGs, so feel free to check/update the proposed SVG. I tried to uniformly align the circles with the 16x16 grid to ensure that rasterization produces as sharp as possible results.

Copy link
Contributor

Test Results

 1 824 files  ±0   1 824 suites  ±0   1h 39m 56s ⏱️ - 2m 37s
 7 925 tests ±0   7 696 ✅  - 1  228 💤 ±0  1 ❌ +1 
23 862 runs  ±0  23 113 ✅  - 1  748 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 0c3dc18. ± Comparison against base commit 710a26a.

@sratz
Copy link
Member

sratz commented May 19, 2025

Note: I am not an expert in creating SVGs, so feel free to check/update the proposed SVG. I tried to uniformly align the circles with the 16x16 grid to ensure that rasterization produces as sharp as possible results.

Hmm, it does look different than the old .png. It's almost like it's squares, not circles.

@sratz
Copy link
Member

sratz commented May 19, 2025

That icon was quite the mess:

Let's start at the source: I tried to re-create the PNGs that have been used over the last years as closely as possible in eclipse-platform/eclipse.platform.images#147 and re-rendered everything.

@BeckerWdf
Copy link
Contributor

Can I help here or did you sort this out in the meantime?

sratz added a commit to sratz/eclipse.platform.images that referenced this pull request May 19, 2025
There were several issues with the view_menu icon:
- The checked in SVG files were not matching the PNG files which have
  been used so far.
- The disabled PNGs were not actually disabled, but copies of the
  enabled ones.
- A @2x.png icon was still the old chevron icon.

Fix:
- Use SVG file from
  eclipse-platform/eclipse.platform.ui#2995,
  clean up metadata and exactly align on pixel boundaries
  and (half) integer stroke width.
- Delete disabled SVG files and let the renderer derive the disabled
  PNG files.
- Re-render everything to PNG.
sratz added a commit to eclipse-platform/eclipse.platform.images that referenced this pull request May 19, 2025
There were several issues with the view_menu icon:
- The checked in SVG files were not matching the PNG files which have
  been used so far.
- The disabled PNGs were not actually disabled, but copies of the
  enabled ones.
- A @2x.png icon was still the old chevron icon.

Fix:
- Use SVG file from
  eclipse-platform/eclipse.platform.ui#2995,
  clean up metadata and exactly align on pixel boundaries
  and (half) integer stroke width.
- Delete disabled SVG files and let the renderer derive the disabled
  PNG files.
- Re-render everything to PNG.
@sratz
Copy link
Member

sratz commented May 19, 2025

Closing in favor of #2994.

@sratz sratz closed this May 19, 2025
@HeikoKlare HeikoKlare deleted the correct-view_menu-svg branch May 19, 2025 14:50
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.

3 participants