If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[Flatfish] No vibrating motor, "Vibrate" option should be removed

NEW
Unassigned

Status

Firefox OS
Gaia
4 years ago
3 years ago

People

(Reporter: ericcc, Unassigned)

Tracking

(Depends on: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [flatfish][TCP=polish] [p=13])

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
GAIA commit e314d089d988f0eb1bef2404858c60b734dc4130

$GAIA_DISTRIBUTION_DIR=distribution_tablet make reset-gaia


1. Open "Clock".
2. Alarm tab, tap "New Alarm" on the top right.
3. <ISSUE> Vibrate option existed.
4. Timer tab, <ISSUE> Vibrate option existed.
(Reporter)

Updated

4 years ago
Flags: needinfo?(frlee)
Is there a tag to detect whether a device supports vibration?
(Reporter)

Comment 2

4 years ago
Created attachment 8367049 [details]
NewAlarmVibrate.png
(Reporter)

Comment 3

4 years ago
Created attachment 8367050 [details]
TimerVibrate.png

Comment 4

4 years ago
flatfish doesn't support Vibration from HW perspective, so vibrate option needs to be removed.
Flags: needinfo?(frlee)
Whiteboard: [Flatfish only] → [Flatfish only][developer+]
This will need some system work; presently there isn't a flag exposed to JavaScript to determine if a vibration motor is present or not. Sadly, the `navigator.vibrate()` API appears to be designed[1] to silently do nothing if no motor exists, so this would need some sort of behavior change or alternate feature-detect at the system level.

[1] https://mxr.mozilla.org/mozilla-central/source/dom/interfaces/base/nsIDOMNavigator.idl?rev=b405f493e834#80
Whiteboard: [Flatfish only][developer+] → [Flatfish only][developer+] [p=13]
Depends on: 971729

Updated

3 years ago
Whiteboard: [Flatfish only][developer+] [p=13] → [flatfish][TCP] [p=13]

Updated

3 years ago
Whiteboard: [flatfish][TCP] [p=13] → [flatfish][TCP=polish] [p=13]
There is also an option in "Settings" -> "Keyboards" -> "Built-in Keyboard" -> "Vibration" that should be removed.

Comment 7

3 years ago
And there is also an option in the Settings app - Personalization - Sound.

So, I changed the summary and component.
Component: Gaia::Clock → Gaia
Summary: [Flatfish][Clock] No vibrating alert motor, "Vibrate" option should be removed from Clock → [Flatfish] No vibrating motor, "Vibrate" option should be removed
You need to log in before you can comment on or make changes to this bug.