Alarm sounds vs Calendar sounds
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Canonical System Image |
High
|
Unassigned | ||
| | indicator-datetime (Ubuntu) |
High
|
Charles Kerr | ||
| | indicator-datetime (Ubuntu RTM) |
Undecided
|
Unassigned | ||
Bug Description
As discussed with rsalveti and pmcgowan in #phablet on 14 Jan
There are two issues in the datetime notification code that need to be changed.
1. Right now we only set role=alarm if the sound is looping; ie if it's an alarm clock event rather than calendar event.
Set calendar event role to alert
2. Calendar events shouldn't play in silent mode. We can test for silent mode on the bus, http://
Related branches
- Antti Kaijanmäki (community): Approve on 2015-01-23
- PS Jenkins bot: Approve (continuous-integration) on 2015-01-22
-
Diff: 281 lines (+128/-14)6 files modifiedinclude/notifications/sound.h (+1/-1)
src/CMakeLists.txt (+3/-0)
src/com.ubuntu.touch.AccountsService.Sound.xml (+42/-0)
src/snap.cpp (+57/-6)
src/sound.cpp (+12/-7)
tests/manual (+13/-0)
- Antti Kaijanmäki (community): Approve on 2015-01-21
-
Diff: 281 lines (+128/-14)6 files modifiedinclude/notifications/sound.h (+1/-1)
src/CMakeLists.txt (+3/-0)
src/com.ubuntu.touch.AccountsService.Sound.xml (+42/-0)
src/snap.cpp (+57/-6)
src/sound.cpp (+12/-7)
tests/manual (+13/-0)
| Pat McGowan (pat-mcgowan) wrote : | #1 |
| Changed in canonical-devices-system-image: | |
| importance: | Undecided → High |
| milestone: | none → ww05-2015 |
| status: | New → Confirmed |
| Changed in indicator-datetime (Ubuntu): | |
| importance: | Undecided → High |
| Ricardo Salveti (rsalveti) wrote : | #2 |
From bug 1391501 it seems we really should be using the role alert, as calendar events are not really alarms. That would also mean that the alert volume (default when nothing is playing) will also affect the calendar events, which is also good.
| description: | updated |
| Changed in indicator-datetime (Ubuntu): | |
| status: | Triaged → In Progress |
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → Fix Released |
| Launchpad Janitor (janitor) wrote : | #3 |
This bug was fixed in the package indicator-datetime - 13.10.0+
---------------
indicator-datetime (13.10.
[ Charles Kerr ]
* Don't play calendar alarm sounds if Silent Mode is enabled. Use the
alarm sound role for both clock and calendar alarms. (LP: #1410874)
[ Ubuntu daily release ]
* New rebuild forced
-- Ubuntu daily release <email address hidden> Thu, 22 Jan 2015 22:25:52 +0000
| Changed in indicator-datetime (Ubuntu RTM): | |
| status: | New → Fix Released |
| Launchpad Janitor (janitor) wrote : | #4 |
This bug was fixed in the package indicator-datetime - 13.10.0+
---------------
indicator-datetime (13.10.
[ Charles Kerr ]
* Don't play calendar alarm sounds if Silent Mode is enabled. Use the
alarm sound role for both clock and calendar alarms. (LP: #1410874)
-- Ubuntu daily release <email address hidden> Wed, 28 Jan 2015 19:16:10 +0000
| Changed in indicator-datetime (Ubuntu): | |
| status: | In Progress → Fix Released |


Queue this up as new sync improvements will make calendar use more widespread