Skip to content

fix(ui): template_obj.svg was actually a png#1001

Merged
sebthom merged 1 commit intoeclipse-tm4e:mainfrom
FlorianKroiss:wrong-file-type
Mar 24, 2026
Merged

fix(ui): template_obj.svg was actually a png#1001
sebthom merged 1 commit intoeclipse-tm4e:mainfrom
FlorianKroiss:wrong-file-type

Conversation

@FlorianKroiss
Copy link
Copy Markdown
Contributor

@FlorianKroiss FlorianKroiss commented Mar 22, 2026

The icon added in #988 was added as .svg but is actually a png, see https://github.com/eclipse-tm4e/tm4e/blob/main/org.eclipse.tm4e.ui/icons/full/obj16/template_obj.svg which GitHub cannot render.

Using file org.eclipse.tm4e.ui/icons/full/obj16/template_obj.svg reveals that the file is actually a PNG:

org.eclipse.tm4e.ui/icons/full/obj16/template_obj.svg: PNG image data, 16 x 16, 8-bit/color RGBA, non-interlaced

This PR replaces the image with the svg from https://github.com/eclipse-platform/eclipse.platform.images/blob/master/org.eclipse.images/eclipse-svg/eclipse.platform.common/bundles/org.eclipse.jdt.doc.user/images/org.eclipse.jdt.ui/obj16/template_obj.svg

@github-actions github-actions bot added the bug label Mar 22, 2026
@FlorianKroiss
Copy link
Copy Markdown
Contributor Author

FlorianKroiss commented Mar 22, 2026

For some reason, GitHub still fails to render the new svg, but opening the file directly using "view file" works

@sebthom sebthom merged commit 25c9a99 into eclipse-tm4e:main Mar 24, 2026
23 checks passed
@sebthom
Copy link
Copy Markdown
Member

sebthom commented Mar 24, 2026

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants