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

fix linux appimage icons #258

Merged
merged 1 commit into from Jan 16, 2019
Merged

Conversation

lianghongzhuo
Copy link
Contributor

In Linux appimage, it generates icons at ~/.local/share/icons/hicolor/0x0/apps for my system.
This will let the system failed to show the app icon.
By creating a folder at ~/notable/resources/icon/icons the appimage will auto-generate all suitable size icons.

@fabiospampinato
Copy link
Member

@lianghongzhuo do you know what this works? We are still only providing the png icon to the builder, it sounds like a problem that should be fixed upstream.

Also having icons inside icon is super weird, I'd avoid doing that. Does pointing to the icon directory work then?

@lianghongzhuo
Copy link
Contributor Author

@fabiospampinato You are right, use icon folder directory also works.

@lianghongzhuo
Copy link
Contributor Author

Here is the explain:

Linux icon set will be generated automatically based on the macOS icns file or common icon.png.

Or you can put them into the build/icons directory if you want to specify them yourself. The filename must contain the size (e.g. 32x32.png) of the icon). Recommended sizes: 16, 24, 32, 48, 64, 96, 128, 256 (or just 512).

@lianghongzhuo
Copy link
Contributor Author

My guess is electron-builder only take build/icons as the default icon location setting. In this project, we use a custom folder which is resources/icon/. This may be the reason case this issue. So the way to solve is either speicify a folder contain icon.png in it. Or specify the file name with the size as the file name (eg: 32x32.png)

@fabiospampinato
Copy link
Member

@lianghongzhuo awesome, thanks!

@fabiospampinato fabiospampinato added this to the Next milestone Jan 10, 2019
@mikaelmello
Copy link
Contributor

Hopeful for this one, as the icon does not currently work for me

@fabiospampinato
Copy link
Member

Thanks @lianghongzhuo, I've also mentioned the problem to electron-builder here electron-userland/electron-builder#3609

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

Successfully merging this pull request may close these issues.

None yet

3 participants