christophgysin / pasystray

PulseAudio system tray
GNU Lesser General Public License v2.1
439 stars 43 forks source link

Mouse middle click doesn't work #152

Open YanDoroshenko opened 2 years ago

YanDoroshenko commented 2 years ago

Looks like the essence of the tray button has changed since 0.8.0 and middle mouse click doesn't mute/unmute sound. Is it possible to bring the middle click back? Thanks.

YanDoroshenko commented 2 years ago

I've done a bit of poking around and it seems that left or right mouse click just opens the menu (ignoring the modifiers) while middle click does nothing at all.

aimixess commented 2 years ago

I tried all of these on tray icon and nothing work for me :

  • ctrl + left-click: start pavucontrol
  • middle-click / alt + left-click: toggle mute default sink
  • ctrl + middle-click / ctrl + alt + left-click: toggle mute default source
  • ctrl + scroll: adjust the default source volume
Abastro commented 2 years ago

Think I am also suffering from this issue. Does the tray support activate/secondary activate from status-notifier-item?

aimixess commented 2 years ago

I just test again with version 0.8.0 and all is working fine if it is launched with symbolic icons. @Abastro, what is your DE/WM ? Did you tried with another one ?

Abastro commented 2 years ago

Okay, let me try and check again

YanDoroshenko commented 2 years ago

I've just run the code from master and that's what I'm seeing: image

New on click: image

Old on click: image

Plus middle click on the old one works as a mute, while the new one does not react to middle click.

Mouse scroll changes volume on both.

By "old" I mean the Arch Linux package that claims to be 0.8.0 too.

Abastro commented 2 years ago

Hm, somehow it works now, even though I did nothing.

YanDoroshenko commented 2 years ago

OK, updates:

"Old" vs "new" is GtkStatusIcon vs AppIndicator, middle click doesn't work on appindicator and logs the following error:

gdk_window_thaw_toplevel_updates: assertion 'window->update_and_descendants_freeze_count > 0' failed
Abastro commented 2 years ago

Just checked the code. https://github.com/christophgysin/pasystray/blob/master/src/systray_impl.c The appindicator version does not implement anything other than the menu and scroll, it seems. Perhaps that's why it has such dependent behavior, now for me middle click does not work again.