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

Tray icon left-click opens context menu (libappindicator) #952

Closed
2 tasks done
LRitzdorf opened this issue Apr 27, 2022 · 2 comments
Closed
2 tasks done

Tray icon left-click opens context menu (libappindicator) #952

LRitzdorf opened this issue Apr 27, 2022 · 2 comments
Labels
Bug Something isn't working More Info Needed Further information is requested YTMD V1 Issue or Feature is already fixed in YTMD V2 / No longer applicible in YTMD V2 Source code

Comments

@LRitzdorf
Copy link

Consent

  • I verified that there is no open issue for the same subject.
  • I understand that YTMDesktop have NO affiliation with Google or YouTube

Current Behavior

Without libappindicator installed, a left-click on the system tray icon reopens the app, as expected.

If libappindicator is installed, even as a dependency for an unrelated package, left-clicks (as well as right-clicks) now open the context menu.

Expected Behavior

With libappindicator installed, left-clicking the tray icon should continue to open the app as normal.

Steps To Reproduce

  1. Install YTMDesktop and libappindicator.
  2. Open YTMDesktop and left-click the system tray icon.
  3. The context menu opens, rather than the app itself.

YTMDesktop

1.14.1

OS

Arch Linux

OS Version

Latest

Arch

x64

Installation way

AUR

Anything else?

This actually appears to be an underlying issue in Electron, described in electron/14941. However, there should be workarounds available.

One particularly interesting method to disable this is recorded in a Bodhi Linux forum post, and I can confirm that it continues to work on Discord (which is also Electron-based). Since Discord installs to /opt/discord, at least on Arch, placing empty files there as described in the forum post works.

If YTMDesktop users would like to do the same, this should be possible, provided we know where YTMDesktop's installation root directory is located. Even if YTMDesktop itself is not updated to fix this issue, motivated users could do so themselves with this method. So, question: where exactly is the installation root for YTMDesktop?

@Alipoodle Alipoodle added Bug Something isn't working More Info Needed Further information is requested labels Apr 27, 2022
@LRitzdorf
Copy link
Author

Looks like this has been resolved upstream! I can't personally verify that it works, since the AUR package for Electron 13 is currently broken for me, but this is a good sign, at least.

See electron/electron#14941 and electron/electron#36333

@Alipoodle Alipoodle added the YTMD V1 Issue or Feature is already fixed in YTMD V2 / No longer applicible in YTMD V2 Source code label Aug 1, 2023
@Alipoodle
Copy link
Member

Closing issue due to no longer supporting YouTube Music Desktop v1 + Release of YouTube Music v2.
Please re-open this issue if this is still occurring in the latest build of the application, however as a re-write has been done on the entire project most issues will be fixed, no longer a feature or re-worked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working More Info Needed Further information is requested YTMD V1 Issue or Feature is already fixed in YTMD V2 / No longer applicible in YTMD V2 Source code
Projects
None yet
Development

No branches or pull requests

2 participants