Merge lp:~ted/indicator-sound/extreme-volume-warning into lp:indicator-sound/15.04
Proposed by
Ted Gould
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Charles Kerr | ||||||||
Approved revision: | 485 | ||||||||
Merged at revision: | 461 | ||||||||
Proposed branch: | lp:~ted/indicator-sound/extreme-volume-warning | ||||||||
Merge into: | lp:indicator-sound/15.04 | ||||||||
Prerequisite: | lp:~ted/indicator-sound/synchronous-notification | ||||||||
Diff against target: |
263 lines (+139/-15) 4 files modified
src/service.vala (+16/-0) src/sound-menu.vala (+39/-1) src/volume-control.vala (+66/-14) tests/manual (+18/-0) |
||||||||
To merge this branch: | bzr merge lp:~ted/indicator-sound/extreme-volume-warning | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Charles Kerr (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+240201@code.launchpad.net |
This proposal supersedes a proposal from 2014-10-13.
Commit message
Warn on high audio levels when using headphones
Description of the change
Both a notification change and a menu item for showing that there is high volume.
To post a comment you must log in.
Ted, I think you can drop this branch. For fixing LP: #1373404 I've the more fitting solution with a patch to lp:unity-settings-daemon, which is the one process triggering volume- notifications upon hardware multi-media-keys already.
My branch lp:~macslow/unity-settings-daemon/max-volume-notification-warning does also fully comply with the dedicated design for the hig-volume-warning.