[reboot] When tapping an alarm sound that is already selected it incorrectly toggles
Bug #1360677 reported by
Victor Thompson
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu Clock App |
Fix Released
|
Low
|
Nekhelesh Ramananthan |
Bug Description
Currently the clock reboot in the app store allows the user to tap a sound that is already selected, causing the sound to appear as though it has been deselected--leaving it seem as though no sounds have been selected.
To reproduce:
1. Swipe up from the bottom to go to the Alarms page
2. Tap the plus (+) icon in the upper right to add a new alarm
3. Tap the Sound list item to change the sound for the alarm
4. Find "Suru arpeggio" and tap it. Note that it goes from checked to unchecked.
5. Tap the back button. Note that "Suru arpeggio" is still the chosen sound, as expected.
Expected results: Step #4 does not allow the item to appear as though it has been deselected.
Related branches
lp://staging/~nik90/ubuntu-clock-app/fix-alarm-sound-toggle-bug
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
- Victor Thompson: Approve
-
Diff: 41 lines (+9/-3)2 files modifiedapp/alarm/AlarmSound.qml (+7/-3)
debian/changelog (+2/-0)
description: | updated |
Changed in ubuntu-clock-app: | |
status: | New → Confirmed |
milestone: | none → rtm |
importance: | Undecided → Low |
Changed in ubuntu-clock-app: | |
status: | Confirmed → In Progress |
assignee: | nobody → Nekhelesh Ramananthan (nik90) |
Changed in ubuntu-clock-app: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Hmm I am not too sure how to fix this. In upstream Qt checkbox, there is a property called exclusiveGroup [1] which allows checkboxes to know they are in group. So only one of them can be checked at any time. I am not sure if the SDK component has this feature. If it does, then this bug would be so easy to fix.
But I will investigate further and see how to fix this.
[1] http:// qt-project. org/doc/ qt-5/qml- qtquick- controls- checkbox. html#exclusiveG roup-prop