Skip to content
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

Changed icon size from 1080x1080 to 512x512 #643

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

thomas-phillips
Copy link

@thomas-phillips thomas-phillips commented Jun 6, 2024

The icon file size was changed to 512x512 as recommended in the electron builder's documentation regarding icon size.

This addresses the #600 issue regarding broken icon size in the Debian 12 LXDE desktop environment.

The icon size electron builder recommends for all platforms is 212x212 or 512x512.

https://www.electron.build/icons.html

20240606_15h46m15s_grim
20240606_15h46m26s_grim

The icon file size was changed to 512x512 as recommended in electron
builder's documentation regarding icon size
@Vendicated
Copy link
Member

Vendicated commented Jun 6, 2024

did you mean to replace the icon in build/ instead?

@thomas-phillips
Copy link
Author

did you mean to replace the icon in build/ instead?

Changing the size of the icon in build/ unfortunately doesn't fix the issue. Could it be as icons are sourced from static/ here that the electron-builder is ignoring icons in build/ for linux builds

@Vendicated
Copy link
Member

the icon in static is only used for tray, nothing else

the icon used in build is the one used as application icon

@thomas-phillips
Copy link
Author

That makes sense, would it be okay to reduce the tray icon size?

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.

2 participants