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 641886 - Volume should go up to 150%
Volume should go up to 150%
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
2.91.x
Other Linux
: Normal normal
: ---
Assigned To: Giovanni Campagna
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2011-02-09 01:11 UTC by Bastien Nocera
Modified: 2011-07-13 19:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
volume: Increase maximum by 50% for outputs with decibel support (1.60 KB, patch)
2011-02-09 20:19 UTC, drago01
none Details | Review
volume: Increase maximum by 50% for outputs with decibel support (2.20 KB, patch)
2011-02-09 20:23 UTC, drago01
needs-work Details | Review
volume: Increase maximum by 50% for outputs with decibel support (3.72 KB, patch)
2011-02-12 13:45 UTC, drago01
none Details | Review
volume: Increase maximum by 50% for outputs with decibel support (3.77 KB, patch)
2011-02-12 14:38 UTC, drago01
committed Details | Review

Description Bastien Nocera 2011-02-09 01:11:23 UTC
Similarly to bug 602494 (volume applet) and bug 631030 (media-keys).

Volume should go up to 150% if the sound card used as output has decibel volume support (eg. if gvc_mixer_stream_get_can_decibel() returns TRUE).
Comment 1 drago01 2011-02-09 20:19:32 UTC
Created attachment 180508 [details] [review]
volume: Increase maximum by 50% for outputs with decibel support

Volume should go up to 150% if the sound card used as output has
decibel volume support.
Comment 2 drago01 2011-02-09 20:23:04 UTC
Created attachment 180509 [details] [review]
volume: Increase maximum by 50% for outputs with decibel support

Volume should go up to 150% if the sound card used as output has
decibel volume support.
Comment 3 Owen Taylor 2011-02-10 19:10:55 UTC
Giovanni - can you review this?
Comment 4 Giovanni Campagna 2011-02-12 11:51:06 UTC
Review of attachment 180509 [details] [review]:

You also need to modify _mutedChanged and _volumeChanged to use a different proportion, or the slider will look maximized when setting a volume louder than 100% using the media keys.
Comment 5 drago01 2011-02-12 13:45:51 UTC
Created attachment 180720 [details] [review]
volume: Increase maximum by 50% for outputs with decibel support

Volume should go up to 150% if the sound card used as output has
decibel volume support.

---
*) Handle _mutedChanged and _volumeChanged too
Comment 6 drago01 2011-02-12 14:38:59 UTC
Created attachment 180722 [details] [review]
volume: Increase maximum by 50% for outputs with decibel support

Volume should go up to 150% if the sound card used as output has
decibel volume support.

---
*) Avoid code duplication
Comment 7 Giovanni Campagna 2011-02-17 17:04:35 UTC
Review of attachment 180722 [details] [review]:

If get_can_decibel does not change without the stream changing underneath (which recreates the Slider), then this patch should work fine.
Comment 8 drago01 2011-02-17 17:08:55 UTC
(In reply to comment #7)
> Review of attachment 180722 [details] [review]:
> 
> If get_can_decibel does not change without the stream changing underneath
> (which recreates the Slider), then this patch should work fine.

It shouldn't.
Comment 9 drago01 2011-02-17 17:10:04 UTC
Attachment 180722 [details] pushed as bdd805a - volume: Increase maximum by 50% for outputs with decibel support
Comment 10 Giulio Fidente 2011-07-13 19:03:58 UTC
hi, I'm the owner of a sound card which provides gvc_mixer_stream_get_can_decibel()

I'm running gnome-shell 3.0.2 and, as per your patch, the volume slider on the top bar volume goes up to 150% if you bring it to its maximum level (eg. using the mouse scroll)

the problem is that when the slider goes past 100% it is actually distorting the sound output

the problem has been reported with the bug #649411 ; I think the slider SHOULD NOT go over 100% by default. It is ok to have some overdrive for laptops but we SHOULD NOT distort the sound by default ... while at the moment, after a fresh install, if you ramp up the volume to its maximum level you are actually distorting the output

apparently the two bug reports contraddict each other, what I'm asking is: why the colume should go up to 150% if the sound card used as output has decibel volume support (eg. if gvc_mixer_stream_get_can_decibel() returns TRUE). ?

thanks for listening