-
Notifications
You must be signed in to change notification settings - Fork 96
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
ElectronMail MacOS icon problem #185
Comments
I confirm the issue. Thanks for the report. |
Although the icon is being implicitly set for Linux and Windows there is something wrong on macOS. Chrome since v22 supports the explicitly specified custom icon for notifications, so going to try this option first of all. |
I am going to try it now |
No, the icon is still incorrect. |
Worked well for me on macOS Catalina 10.15. At the moment I don't have another macOS system to test the stuff as well as don't have other ideas of the potential fix. |
I have this issue when the app is in a folder. The icon will show normally when you click on the folder, but not when the folder is closed. ElectronMail v3.7.0 |
Can someone confirm that v3.7.1 is affected? That version has got @electron 5 => 6 update and so maybe the issue got resolved upstream. |
I still have the folder icon problem with v3.7.1 |
I'm running v3.7.1 on my MacBook Pro, macOS Mojave 10.14.6, I also have the issue. |
@vladimiry I have the same issue. It looks like that in notifications when I get a new message. |
This issue doesn't happen anymore for me on macOS Catalina 10.15.1, ElectronMail v3.8.0. Not sure which update fixed it. |
Still having this on 10.14.6. |
This is an upstream issue as the app doesn't explicitly render the icon but only specifies the PNG image. So the only possible solution is updating @electron version or macOS version. We got the message above that updating to Catalina helps. I work on upgrading to @electron version 8 hoping that it will resolve the issue for everyone but there are still blocking upstream issues. |
Can someone who still experiences the issue test this build? Changes:
So the build uses a dynamic/DataURL image instead of the static file, so maybe it will work around the original issue. PS don't use this build on a regular basis as it's a little buggy (at least don't edit/add the accounts to the app). |
Updated the build link in previous message, it's now safe to use it on a daily basis. |
I'm closing the issues since as I wrote before it's the upstream issue and all I could do is updating the Electron versions in time (Electron version 8 is going to be released quite soon). I'd recommend trying the recent release. There are now 3 packages provided for specific macOS versions and it increases chances for better compatibility. |
The ElectronMail icon on notifications in MacOS is different from the main being used.
Here is a printscreen to this defective icon shown.
The text was updated successfully, but these errors were encountered: