Clock app buttons vibrate when "Other vibrations" is off
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu Clock App |
New
|
Undecided
|
Unassigned | ||
| ubuntu-ui-toolkit (Ubuntu) |
New
|
Undecided
|
Unassigned | ||
Bug Description
Ubuntu 15.04 r82
1. In System Settings > "Sound", turn off "Other vibrations".
2. Launch the Clock app.
3. Tap "Add City".
4. Tap the back button in the header.
What happens:
3. The phone vibrates.
4. The phone vibrates.
What should happen:
3. The phone doesn't vibrate.
4. The phone doesn't vibrate.
This might be the result of confusion with the Clock app's "Vibration" setting. It's understandable that the Clock app might want its alarms to vibrate, but that does not mean its buttons should regardless of System Settings. Probably the setting should be relabelled "Vibrate on alarm" or similar.
Separately, this is a toolkit issue. The Clock app appears to be using a standard header control, but somehow even the header's buttons are not following the "Other vibrations" setting. If following that setting is something app developers have to remember to do, inevitably sometimes they will forget. They shouldn't have to remember.
[Originally mentioned by Victor Thompson in bug 1412622.]
| Nekhelesh Ramananthan (nik90) wrote | #1 |
From my testing, it seems QML components derived from the SDK AbstractButton [1] do not seem to obey the "Other vibrations" settings. Clock app does not have any individual settings that overrides the system's vibrate setting.
In the clock app settings page, we do have a settings option called "Vibration", but thats meant for Alarms, that is when an alarms rings, a snap notification is shown along with the alarm ringtone played and vibration. That vibration is what the settings option is meant to enable/disable.
Those are two different use-cases.
Also the issue of AbstractButton not obeying the system "Other Vibrations" settings was reported previously at bug 1466659. As such I am marking this bug a duplicate. If you want the string "Vibration" to be changed to "Vibrate on alarm", please create a new bug report to track that issue.
[1] https:/