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 790281 - Add setting about whether to allow sound volume > 100%
Add setting about whether to allow sound volume > 100%
Status: RESOLVED FIXED
Product: gsettings-desktop-schemas
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gsettings-desktop-schemas-maint
gsettings-desktop-schemas-maint
Depends on:
Blocks: 710424 790988
 
 
Reported: 2017-11-13 08:45 UTC by Didier Roche
Modified: 2017-11-29 16:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
[PATCH] Allow pulseaudio to amplify volume key (1.09 KB, patch)
2017-11-13 08:46 UTC, Didier Roche
none Details | Review
[PATCH] schemas: Add a gsettings key to be able to set volume above 100% (1.05 KB, patch)
2017-11-14 10:35 UTC, Didier Roche
accepted-commit_now Details | Review
schemas: Add setting about whether to allow volume above 100% (1.05 KB, patch)
2017-11-14 10:54 UTC, Bastien Nocera
committed Details | Review

Description Didier Roche 2017-11-13 08:45:27 UTC
This is the new schema for https://bugzilla.gnome.org/show_bug.cgi?id=710424, where the general design discussion is.
Comment 1 Didier Roche 2017-11-13 08:46:37 UTC
Created attachment 363485 [details] [review]
[PATCH] Allow pulseaudio to amplify volume key
Comment 2 Bastien Nocera 2017-11-14 10:13:41 UTC
Review of attachment 363485 [details] [review]:

Please follow the commit message pattern used by previous commits.

::: schemas/org.gnome.desktop.sound.gschema.xml.in
@@ +17,3 @@
       <description>Whether to play sounds on input events.</description>
     </key>
+    <key type="b" name="allow-amplified-volume">

"Amplified volume" doesn't mean "above 100%". Be clear about the fact that this is a software solution to a hardware problem, and the risks it causes.

@@ +19,3 @@
+    <key type="b" name="allow-amplified-volume">
+      <default>false</default>
+      <summary>Allow pulseaudio to amplify volume</summary>

There's no need to mention PulseAudio in the schema.

@@ +20,3 @@
+      <default>false</default>
+      <summary>Allow pulseaudio to amplify volume</summary>
+      <description>Whether the volume slider and multimedia keys allows setting the volume above 100%.</description>

Just those 2?
Comment 3 Didier Roche 2017-11-14 10:35:51 UTC
Created attachment 363571 [details] [review]
[PATCH] schemas: Add a gsettings key to be able to set volume above 100%

Thanks for the review! I tried to reformulate and hope this fits more.

I'll update the other patch to read the new schema once merged.
Comment 4 Bastien Nocera 2017-11-14 10:39:16 UTC
Review of attachment 363571 [details] [review]:

Looks good.
Comment 5 Bastien Nocera 2017-11-14 10:54:25 UTC
Created attachment 363572 [details] [review]
schemas: Add setting about whether to allow volume above 100%
Comment 6 Bastien Nocera 2017-11-14 10:58:25 UTC
Attachment 363572 [details] pushed as da7eab3 - schemas: Add setting about whether to allow volume above 100%