Canonical System Image

UI volume warnings need revision

Bug #1481913 reported by John McAleely
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Critical
Alejandro J. Cura
Canonical System Image ww34-2015 "ota6"
indicator-sound (Ubuntu)
Fix Released
Critical
Xavi Garcia

Bug Description

As raised in OEM bug #1474364 (the title there is poor), the warning scheme used by Ubuntu does not currently follow the letter of the EU regulations for mobile devices (it seems to follow the spirit).

Design have now reviewed the regulations (bug #1480912), and propose:

https://wiki.ubuntu.com/Sound#limits

This is the diff: https://wiki.ubuntu.com/Sound?action=diff&rev2=146&rev1=145

The UI notification scheme needs to be updated to match the new design.

Tags: audio

Related branches

lp:~unity-api-team/indicator-sound/lp-1481913-revised-ui-volume-warnings
PS Jenkins bot (community): Approve (continuous-integration)
Ted Gould (community): Approve
Diff: 421 lines (+175/-85)
6 files modified
data/com.canonical.indicator.sound.gschema.xml (+39/-0)
src/service.vala (+117/-80)
src/volume-control-pulse.vala (+9/-1)
tests/CMakeLists.txt (+3/-1)
tests/media-player-user.cc (+2/-2)
tests/volume-control-test.cc (+5/-1)
lp:~charlesk/indicator-sound/lp-1481913-integrate-settings-and-clamp-volume
Antti Kaijanmäki (community): Approve
PS Jenkins bot (community): Approve (continuous-integration)
Diff: 856 lines (+356/-175)
7 files modified
data/com.canonical.indicator.sound.gschema.xml (+58/-34)
src/service.vala (+112/-106)
src/volume-control-pulse.vala (+161/-22)
src/volume-control.vala (+13/-1)
tests/CMakeLists.txt (+0/-2)
tests/notifications-test.cc (+8/-8)
tests/volume-control-mock.vala (+4/-2)
lp:ubuntu/wily-proposed/indicator-sound
Revision history for this message
John McAleely (john.mcaleely) wrote :

I assume the system already has an agreed way to know which is the '85dB level' with the driver. If not, the work in bug #1481916 should include those changes.

Changed in canonical-devices-system-image:
status: New → Confirmed
importance: Undecided → Critical
milestone: none → ww34-2015
Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
assignee: nobody → Alejandro J. Cura (alecu)
Changed in indicator-sound (Ubuntu):
assignee: nobody → Xavi Garcia (xavi-garcia-mena)
status: New → Confirmed
importance: Undecided → Critical
John McAleely (john.mcaleely)
tags: added: audio
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-sound - 12.10.2+15.10.20150807.6-0ubuntu1

---------------
indicator-sound (12.10.2+15.10.20150807.6-0ubuntu1) wily; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.

  [ Charles Kerr ]
  * Revised UI volume warnings to comply with EU requirements. (LP:
    #1481913)

 -- CI Train Bot <email address hidden> Fri, 07 Aug 2015 22:35:58 +0000

Changed in indicator-sound (Ubuntu):
status: Confirmed → Fix Released
Jean-Baptiste Lallement (jibel)
Changed in canonical-devices-system-image:
status: Confirmed → Fix Committed
Revision history for this message
John McAleely (john.mcaleely) wrote :

In extremis, if we can't adjust the driver sucessfully, what value should be modified to move the level at which the warning UI appears? I would like to confirm that if I get the warnings to appear at the 50% point on the indicator slider, we do conform to the regulations (ie that point matches the required output dB/mV).

Revision history for this message
Alejandro J. Cura (alecu) wrote :

The dconf key for this is "high-volume-level", it defaults to 0.75 and can be overridden in the device tarball so each OEM can adjust this value to specific devices.

Details are in the first branch attached to this bug:
https://code.launchpad.net/~unity-api-team/indicator-sound/lp-1481913-revised-ui-volume-warnings/+merge/267393#L8

Revision history for this message
John McAleely (john.mcaleely) wrote :

Thanks alecu - it seems that the code moved on in the second branch, and my experiment suggests the key in question is:

com.canonical.indicator.sound warning-volume-decibels

If I set this to -16.0 (the default is -8.0) the warning appears around the 50% mark of the indicator.

Revision history for this message
Alejandro J. Cura (alecu) wrote :

Ah, my bad. Sorry for pointing at the wrong one.

Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.