Display "no active alarm" while checking

Bug #1381432 reported by Sebastien Bacher
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Clock App
Fix Released
Medium
Nekhelesh Ramananthan

Bug Description

Using rtm 106 on krillin

- configure an alarm
- close the clock application
- start it again
- look at the button of the screen

-> on start the label states "no active alarm" for a second before updating and displaying the correct information, that can be misleading. Tt would be better if it displayed nothing or "checking" while it's looking

Related branches

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Hmm this isn't an easy one to fix for RTM due to some missing signals in the Alarms API.

At the moment, here is the start up sequence of the clock app.

- Main Clock App UI is loaded (clock face, labels, buttons etc)
- Alarm Model Loader starts loading the alarm model
- Bottom Edge Hint (loaded and shown *only* after the alarm model loader is ready)

The issue (I think) is that when the bottom edge hint is shown, the alarm model loader has finished loading the alarm model. *But* the alarm model itself is still in the process of fetching data from EDS. But there is no signal fired in the Alarms API for clock app to know about this. As a result, clock app checks and see that there are no alarms and hence shows "No Active Alarms" temporarily.

On my phone that I dogfood daily, I don't have the calendar app installed which results in a shorter loading time for the alarm model. As such the "No Active Alarms" string is barely shown for at most 200-300 ms. While those who have calendar app installed and configured with google calendar will have longer alarm model loading time.

I will look into this, but cannot get it done before RTM I am afraid.

Changed in ubuntu-clock-app:
importance: Undecided → Medium
status: New → Confirmed
milestone: none → ota-1
Revision history for this message
Zsombor Egri (zsombi) wrote :

You can use modelReset() signal from the AlarmModel to catch when the alarms are available. So you can detect when to show information about the alarms.

Changed in ubuntu-clock-app:
assignee: nobody → Nekhelesh Ramananthan (nik90)
status: Confirmed → In Progress
Changed in ubuntu-clock-app:
milestone: 3.3 → 3.4
Changed in ubuntu-clock-app:
assignee: Nekhelesh Ramananthan (nik90) → nobody
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

Fix committed into lp:ubuntu-clock-app at revision 298, scheduled for release in ubuntu-clock-app, milestone 3.4

Changed in ubuntu-clock-app:
status: In Progress → Fix Committed
Changed in ubuntu-clock-app:
assignee: nobody → Nekhelesh Ramananthan (nik90)
Changed in ubuntu-clock-app:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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