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

No menu icon after updating to 2.8.3-linux1. #522

Closed
K0-RR opened this issue Jun 8, 2021 · 10 comments
Closed

No menu icon after updating to 2.8.3-linux1. #522

K0-RR opened this issue Jun 8, 2021 · 10 comments
Labels
installers/deb Issues related to the Debian installer needs-investigation retest-requested Issue needs to be verified against latest version of app

Comments

@K0-RR
Copy link

K0-RR commented Jun 8, 2021

Installed the latest deb and now I can only launch it from the terminal.

OS: Ubuntu Cinnamon 21.04 x86_64
Kernel: 5.11.0-18-generic

@shiftkey
Copy link
Owner

shiftkey commented Jun 8, 2021

Could there be something wrong/deprecated with the .desktop file we include in the installer that's now removed in Cinnamon and causing it to not be detected by the OS?

@shiftkey shiftkey added the installers/deb Issues related to the Debian installer label Jun 8, 2021
@K0-RR
Copy link
Author

K0-RR commented Jun 8, 2021

As I skipped a few versions I downgraded to 2.8.1 Linux RC3 to check this properly and the icon was there again, then I updated to 2.8.2 Linux RC1 and the icon is no more so probably something is wrong beginning from this version.

@K0-RR
Copy link
Author

K0-RR commented Jul 12, 2021

I'm now on 2.9.0 Linux RC2 and the menu icon is back again. Were there any changes that could have fixed this?

Edit. now on 2.9.0 Linux RC3 and again no menu entry, I really don't know what is happening.

@K0-RR
Copy link
Author

K0-RR commented Jul 15, 2021

It now also doesn't have an icon
image

@shiftkey
Copy link
Owner

Only thing that's changed with 2.9.0-linux3 is the icons were updated in #530 to match the new theme, which I thought would fix a different bug. No changes to path names, just the contents of the images.

@shiftkey
Copy link
Owner

Actually, there was one other change that may be the root cause of this recent regression. I removed this line which was setting the icon at the app window level:

windowOptions.icon = path.join(__dirname, 'static', 'icon-logo.png')

Perhaps I need to put that back, as it turns out this was used by the app at runtime.

@shiftkey
Copy link
Owner

Please test 2.9.0-linux4 to see if that resolves this latest issue.

I'm still stumped a bit about why the menu icon isn't working in Cinnamon, but I'm also not familiar with how Cinnamon changes things for Ubuntu...

@shiftkey shiftkey added the retest-requested Issue needs to be verified against latest version of app label Jul 19, 2021
@K0-RR
Copy link
Author

K0-RR commented Jul 19, 2021

This release resolves both issues, yeah I don't know what is going on.
I will create a virtual machines with cinnamon (Mint and Ubuntu) to test this again but since the Github menu icon is the only one that disappears and appears randomly when updating I doubt it's something with my system.

Also, this gets printed when launching from a terminal but you probably know about it
(node:2389692) electron: The default of contextIsolation is deprecated and will be changing from false to true in a future release of Electron. See https://github.com/electron/electron/issues/23506 for more information

@shiftkey
Copy link
Owner

@RDKRACZ thanks for confirming. Happy to close this out and reopen it if you can spot what might have caused it.

That contextIsolation message is a warning about an upcoming Electron change. I see it locally too, might be able to mute it but we'll see if it breaks the app and needs more work before Electron 12 is adopted upstream.

@K0-RR
Copy link
Author

K0-RR commented Jul 20, 2021

I will create virtual machines with cinnamon (Mint and Ubuntu) to test this again...

Not able to reproduce on a virtual machine, I know this issue is resolved but I'm so confused why this was happening.
Additionally I created a Linux Mint Xfce machine to test #559 with 2.9.0-linux4 and can confirm there is no icon on the taskbar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
installers/deb Issues related to the Debian installer needs-investigation retest-requested Issue needs to be verified against latest version of app
Projects
None yet
Development

No branches or pull requests

2 participants