Mouse release does not activate playback control buttons

[David Tolnay]

indicator-sound 0.6.6.1-0ubuntu3, Ubuntu 11.04
indicator-sound 12.10.2+14.04.20140401-0ubuntu1, Ubuntu 14.04

In maverick's sound menu, it was possible to activate the banshee or rhythmbox play/pause, back, and forward controls by pressing the mouse on the indicator applet sound icon, dragging to the chosen control with the mouse button held down, and releasing the mouse button. It looks like the sound menu spec at https://wiki.ubuntu.com/SoundMenu is ambiguous as to whether this should activate the controls, but all other menu items in the indicator applet can be activated in this way (as an example, the mute option in the sound menu). The fact that this no longer works for the playback control buttons seems like a bug.

To be clear, click/release on the sound menu icon followed by click/release on the playback buttons works perfectly well. The inconsistency is in the playback buttons being more picky than other menu items, in that they can't be triggered by just a mouse release.

<https://wiki.ubuntu.com/Sound#playback-item>: "Conversely, clicking on a different menu item (or any menu title), moving to a button, and releasing, should activate that button."

[Conor Curran (cjcurran)]

I don't consider this a bug but more like a bug fix. Play controls or menuitems should not be activated if the proceeding press event occurred in an area outside of the control's space.
The Maverick version was buggy in its handling of key events.

Thanks for reporting this but I feel its a non-starter.

[David Tolnay (dtolnay)]

In the past, menuitems have always been activated in this way. They still are (cf. mute option in the sound menu, the applications/places/system menus in GNOME classic, any time you have a context menu). Since that seems to be the precedent, what justifies the playback controls being the single exception?

[David Tolnay (dtolnay)]

Adding bug #785678 as a duplicate.

[Conor Curran (cjcurran)]

I'm not convinced to be honest, awaiting design feedback.

[Matthew Paul Thomas (mpt)]

The playback controls aren't the only menu controls that require mousedown as well as mouseup. Just above them is the volume slider, which behaves the same way. And next door, the calendar in the clock menu, which similarly requires mousedown.

For each of those, we could define that they should work on mouseup without mousedown, like menu items do. Would there be any drawbacks?

[Matthew Paul Thomas (mpt)]

Specification updated. <https://wiki.ubuntu.com/Sound?action=diff&rev2=101&rev1=100>

[Xavi Garcia (xavi-garcia-mena)]

This bug is very very old.

Marking it as invalid to clean up the actual bug list.
If anybody thinks it should be reopened, please feel free to do so

[Matthew Paul Thomas (mpt)]

Reproduced in indicator-sound 12.10.2+14.04.20140401-0ubuntu1, Ubuntu 14.04.

[dobey (dobey)]

Moving to ido where the widget is implemented. The indicator itself doesn't know anything about mouse button press state, menu position or such, as it doesn't render the widgets. That's done on the unity side.