Not able to change "Repeats" setting in AddAlarm page when "Daily" value is set
Bug #1270092 reported by
Stefano Verzegnassi
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu Clock App |
Fix Released
|
Medium
|
Stefano Verzegnassi |
Bug Description
Tested on: Ubuntu 14.04 LTS and Ubuntu Touch 14.04 (r121 on 2012 Nexus 7)
1) Run Clock app
2) Go in the Alarm page
3) Add alarm
4) Set the "Repeats" option selector as "Daily"
5) Try to undo the change
I attached a video showing this issue.
Difficulty: EASY
----- Desired Solution (nik90) -----------
1. Remove the "Daily" option from the alarm options
2. Add a switch called "Repeat" disabled by default. If the Repeat switch is enabled by the user, then the user can choose multiple days on which the alarm repeats on.
Here is a nice mockup by Stefano https:/
Related branches
lp://qastaging/~verzegnassi-stefano/ubuntu-clock-app/fix-bug-1270092
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
- Nekhelesh Ramananthan: Approve
-
Diff: 99 lines (+31/-37)1 file modifiedalarm/AddAlarmPage.qml (+31/-37)
Changed in ubuntu-clock-app: | |
status: | New → Confirmed |
milestone: | none → 1.6 |
tags: | added: hackday |
description: | updated |
Changed in ubuntu-clock-app: | |
status: | Confirmed → Triaged |
importance: | Undecided → Medium |
description: | updated |
Changed in ubuntu-clock-app: | |
status: | Triaged → In Progress |
Changed in ubuntu-clock-app: | |
assignee: | nobody → Stefano Verzegnassi (verzegnassi-stefano) |
Changed in ubuntu-clock-app: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Now I've seen that this bug report is a duplicate.
Anyway, I took a look to the problem, and I notice a thing: if I set a weekly alarm and choose that it occurs from Monday to Sunday, I expect to have set a daily alarm. In fact, when I go back to the Alarm page, the app says that I've set a daily alarm.
So I think that "Repeats daily" option is just a duplicate of a specific case for "weekly".
I made a quick mock-up, and I added an "Every day" option in the AlarmDays dialog, which change the state of all the switchers.
It's attached to this comment.
I've tested it in Ubuntu SDK and it works as expected (no problem with OptionSelector).