After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 461944 - pressing the volume icon in full screen shuts down the sound
pressing the volume icon in full screen shuts down the sound
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: Other
2.11.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
: 504161 526880 532115 540669 561864 584231 602340 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2007-07-30 20:44 UTC by Baptiste Mille-Mathias
Modified: 2009-11-20 02:25 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Attachments
gtk-dont-set-scale-to-zero.patch (1.58 KB, patch)
2008-04-09 09:50 UTC, Bastien Nocera
none Details | Review
patch to fix setting volume to zero (1.85 KB, patch)
2009-04-08 17:44 UTC, Evgeny Khrustov
needs-work Details | Review
0001-Bug-461944-pressing-the-volume-icon-in-full-scre.patch (2.26 KB, patch)
2009-06-22 16:22 UTC, Bastien Nocera
committed Details | Review

Description Baptiste Mille-Mathias 2007-07-30 20:44:16 UTC
Hi,

When running totem in fullscreen, if I press the volume button to make the slider appear, the slider goes at the bottom because of the mouse pointer and so the volume is shut down.
Comment 1 Bastien Nocera 2007-07-30 21:11:18 UTC
Which version of Totem?
Comment 2 Baptiste Mille-Mathias 2007-07-30 21:16:29 UTC
oups sorry, 2.19.6
Comment 3 Bastien Nocera 2007-07-30 21:19:43 UTC
Yeah, you're supposed to click-drag the sound. Otherwise, it would be outside the screen, or we'd need to make the cursor jump. If anyone else has a better idea...
Comment 4 Cristian Aravena Romero 2007-12-02 21:09:01 UTC
Function of icon button of sound idem to function of icon of button of youtube.

== Spanish ==
Si el método del botón trabajara de la misma manera que el método del boton de youtube sería genial!!!. : -)

¿Qué falta?
* Que la barra se abriera siempre en el mismo lugar.
* El scroll no sea sensible a la posición del puntero del mouse.

Este bug tambien esta en launchpad.net
https://bugs.edge.launchpad.net/ubuntu/+source/totem/+bug/137041
Saludos!
Comment 5 Bastien Nocera 2007-12-18 10:45:18 UTC
*** Bug 504161 has been marked as a duplicate of this bug. ***
Comment 6 Jonas Jørgensen 2007-12-30 10:32:37 UTC
(In reply to comment #3)
> Yeah, you're supposed to click-drag the sound. Otherwise, it would be outside
> the screen, or we'd need to make the cursor jump. If anyone else has a better
> idea...

Here's a better idea: When the volume button is clicked, the slider opens, but without changing the current volume. That is, only display the control but don't change its value on a simple click, and ignore the rule that the cursor should be positioned over the currently selected spot on the slider.

That, or make the cursor jump. Either way is acceptable, but having the movie you're watching become muted just by clicking the volume button is not.
Comment 7 David Prieto 2008-03-10 14:57:49 UTC
"Yeah, you're supposed to click-drag the sound".

Bastien, that's broken behaviour. Normal volume widgets have that option, but they also let you just click them to get the volume bar. Here, clicking the widget mutes the sound. As for dragging it? It mutes the sound too, until you drag it back up. There's no reason to completely mute the volume when you just want to change it a bit.

I think there are two options to fix this problem:

-Get back the previous volume bar - where the volume was not shown and controlled by a widget, but an horizontal bar.
-Make the fullscreen volume widget work EXACTLY like the volume applet. That's: 1 click brings up the volume bar, then you can drag it.
Comment 8 Bastien Nocera 2008-04-09 09:38:59 UTC
*** Bug 526880 has been marked as a duplicate of this bug. ***
Comment 9 Bastien Nocera 2008-04-09 09:50:13 UTC
Created attachment 108914 [details] [review]
gtk-dont-set-scale-to-zero.patch

Please test this patch. It should send the first click to the volume button if the popup was moved (usually, the current position of the slider is straight under the mouse, as you can see in windowed mode).
Comment 10 Bastien Nocera 2008-05-08 09:06:45 UTC
*** Bug 532115 has been marked as a duplicate of this bug. ***
Comment 11 Philip Ganchev 2008-05-08 11:07:04 UTC
The volume applet is cumbersome because it requires the user to position the pointer precisely over the slider.  It would be more intelligent to move the pointer to the current volume position while the slider is shown, and bring it back to the original position when it hides again.  This should not be hard to implement -- just remember the position of the pointer, and move the pointer twice.  I think I will post a bug for the volume applet too.

The approach in the patch makes the volume control modal -- the behavior depends on the window's position in relation to the screen edge.  That would be surprising and annoying to users.
Comment 12 Pascal Schott 2008-10-27 07:30:25 UTC
This bug is still present in version 2.24
Comment 13 Bastien Nocera 2008-11-01 01:24:44 UTC
That would be because nobody tested the patch...
Comment 14 Pascal Schott 2008-11-04 20:41:03 UTC
What is the way to proceed to test the patch ? I don't know what to do with the attachment.
Comment 15 Darin 2008-11-17 02:50:50 UTC
This doesn't only happen when a program is Maximized/Full-Screened. Even if the Volume Slider has enough space on the screen, and the mouse isn't moved at all, clicking the volume button changes the volume by 2-3%.

When a program like Totem, Banshee, Rhythmbox, etc is Full-Screened or Maximized, it is simply more drastic. If the button is at the top of the screen and volume is low, it changes to high. If the button is at the bottom of the screen, and volume is high, it changes it to very low or mute.

It would be nice to see the Volume Slider not appear until Click Release. It would also be nice to see the volume slider appear either below the button or above the button, depending on space/placement. A Drawer effect would be nice (Panel Drawers/YouTube volume control). Percentage shown would also be nice (Bug 440131). But it shouldn't appear over top of the button you're clicking, or change unexpectedly.

Also, I think these are Duplicates:

Bug 540669
Bug 460733
Bug 327149

Thanks!

- Darin
Comment 16 David Prieto 2008-11-17 05:43:49 UTC
"It would be nice to see the Volume Slider not appear until Click Release"

With that I disagree. Being able to set the volume with just one mouse click is one of my favourite thing in Gnome media players.

Maybe make it not appear until click release if (and only if) it detects that it would collide with a screen border.
Comment 17 Bastien Nocera 2008-11-21 22:21:54 UTC
*** Bug 561864 has been marked as a duplicate of this bug. ***
Comment 18 Sebastien Bacher 2009-02-26 22:01:10 UTC
*** Bug 540669 has been marked as a duplicate of this bug. ***
Comment 19 Evgeny Khrustov 2009-04-08 17:44:43 UTC
Created attachment 132354 [details] [review]
patch to fix setting volume to zero
Comment 20 Evgeny Khrustov 2009-04-08 17:47:09 UTC
Patch working, but i applied him manually, because  gtkscalebutton.c has changed in gtk+-2.14.4. Tested in gnome-mplayer and totem in fullscreen mode. All looks good. actual patch in attach.
Comment 21 Evgeny Khrustov 2009-04-10 16:53:54 UTC
hey...
anyone there?
Please, commit patch
Comment 22 thomas hulard 2009-06-09 20:16:14 UTC
Hi guys,

I just upgrade my Fedora to the 11 version hoping this annoying bug will be gone.

I run Totem 2.26.2 with Gstreamer 0.10.22 and the issue is present.
I don't get the fact the bug as a UNCONFIRMED status as it is pretty easy to reproduce.

Any news from this?
Comment 23 Bastien Nocera 2009-06-09 20:26:05 UTC
(In reply to comment #22)
> I run Totem 2.26.2 with Gstreamer 0.10.22 and the issue is present.
> I don't get the fact the bug as a UNCONFIRMED status as it is pretty easy to
> reproduce.

The status doesn't mean anything, it's bugzilla book-keeping.

> Any news from this?

Read the bug, and try the patch.
Comment 24 Johannes Mockenhaupt 2009-06-22 15:45:20 UTC
I've applied the patch from Evgeny to gtk+2.0_2.16.1-0ubuntu2 (Ubuntu Jaunty) and is works well. Singe click opens the slider now instead of doing a click-and-drag.
Comment 25 Bastien Nocera 2009-06-22 16:19:32 UTC
The reason why the patch is so long compared to the one I submitted is because you made unwarranted changes to the identation.
Comment 26 Bastien Nocera 2009-06-22 16:22:24 UTC
Created attachment 137183 [details] [review]
0001-Bug-461944-pressing-the-volume-icon-in-full-scre.patch

Updated patch for current master.
Comment 27 Bastien Nocera 2009-06-22 16:29:57 UTC
Fixed in master and gtk-2-16

commit b436f5b8a3c8ad0468d2420895fa97dc7c711969
Author: Bastien Nocera <hadess@hadess.net>
Date:   Mon Jun 22 17:19:21 2009 +0100

    Bug 461944 – pressing the volume icon in full screen shuts down the sound
    
    Don't pass the click on the button through to the scale when the
    dock popup will be moved, otherwise we could end up changing the
    sound in unexcepted ways (to zero for vertical popups at the bottom
    of the screen for example).
Comment 28 Milan Bouchet-Valat 2009-06-22 17:55:57 UTC
Thank you so much to both of you for fixing this quite ridiculous bug that has been hitting us for too long! ;-)
Comment 29 Bastien Nocera 2009-07-21 01:46:52 UTC
*** Bug 584231 has been marked as a duplicate of this bug. ***
Comment 30 Bastien Nocera 2009-11-20 02:25:15 UTC
*** Bug 602340 has been marked as a duplicate of this bug. ***