Alarm sounds vs Calendar sounds

Bug #1410874 reported by Charles Kerr
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Unassigned
indicator-datetime (Ubuntu)
Fix Released
High
Charles Kerr
indicator-datetime (Ubuntu RTM)
Fix Released
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://bazaar.launchpad.net/~phablet-team/telephony-service/trunk/view/head:/libtelephonyservice/greetercontacts.cpp has an example: getUserValue("com.ubuntu.touch.AccountsService.Sound", "SilentMode");

Related branches

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

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

Changed in canonical-devices-system-image:
importance: Undecided → High
milestone: none → ww05-2015
status: New → Confirmed
Charles Kerr (charlesk)
Changed in indicator-datetime (Ubuntu):
importance: Undecided → High
Revision history for this message
Ricardo Salveti (rsalveti) wrote :

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
Charles Kerr (charlesk)
Changed in indicator-datetime (Ubuntu):
status: Triaged → In Progress
Changed in canonical-devices-system-image:
status: Confirmed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-datetime - 13.10.0+15.04.20150122~rtm-0ubuntu1

---------------
indicator-datetime (13.10.0+15.04.20150122~rtm-0ubuntu1) 14.09; urgency=low

  [ 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
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-datetime - 13.10.0+15.04.20150128.1-0ubuntu1

---------------
indicator-datetime (13.10.0+15.04.20150128.1-0ubuntu1) vivid; urgency=low

  [ 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
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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