Default alarm time is wrongly set if the current time minutes is a multiple by 5
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu Clock App |
Fix Released
|
Low
|
Bartosz Kosiorek | ||
Bug Description
According to Clock Design the default alarm time shown in the edit alarm page should be taken from the current time and be rounded off to the next multiple of 5.
So for instance, if the current time is 13:26, then the edit alarm page should show 13:30 and so on.
Unfortunately when current time's minute is divisible by 5, like 13:30 for instance, then the alarm is set to 24 hours later.
Steps to reproduce.
1. Wait current time's minute is divisible by 5 (eg. 13:25)
2. Press + button to create alarm
3. Notice that time is "13:25"
4. After alarm creation, the alarm is set for 24 hours
Implementation proposal
1. Wait current time's minute is divisible by 5 (eg. 13:25)
2. Press + button to create alarm
3. Notice that time is "13:30"
4. After alarm creation, the alarm is set for 5 minutes later
Related branches
- Nekhelesh Ramananthan: Approve
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
-
Diff: 30 lines (+4/-3)2 files modifiedapp/alarm/EditAlarmPage.qml (+3/-3)
debian/changelog (+1/-0)
| Changed in ubuntu-clock-app: | |
| milestone: | none → 3.5 |
| summary: |
- Default alarm time is wrongly set, if the current time minutes is - multiply by 5 + Default alarm time is wrongly set if the current time minutes is a + multiple by 5 |
| Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote | #1 |
| Changed in ubuntu-clock-app: | |
| status: | In Progress → Fix Committed |
| description: | updated |
| Changed in ubuntu-clock-app: | |
| status: | Fix Committed → Fix Released |
Fix committed into lp:ubuntu-clock-app at revision 334, scheduled for release in ubuntu-clock-app, milestone 3.5