Closed Bug 1230613 Opened 9 years ago Closed 9 years ago

Long-press to trigger text selection should vibrate

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
All
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox45 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 45
Tracking Flags:
Tracking Status
firefox45 --- fixed

People

(Reporter: antlam, Assigned: capella)

References

Details

Attachments

(1 file)

bug1230613.diff
(deleted), patch
snorp
: review+
Details | Diff | Splinter Review 
It's hard to know when the selection has been "triggered" with the vibration missing now.

Can we add that back?
Attached patch bug1230613.diff (deleted) — — Splinter Review 
Yah, I miss it too !
Assignee: nobody → markcapella
Status: NEW → ASSIGNED
Attachment #8697149 - Flags: review?(snorp)
Comment on attachment 8697149 [details] [diff] [review]
bug1230613.diff

Review of attachment 8697149 [details] [diff] [review]:
-----------------------------------------------------------------

nice!
Attachment #8697149 - Flags: review?(snorp) → review+
Push to try; https://treeherder.mozilla.org/#/jobs?repo=try&revision=41b70bc28d6d

I can't clear the orange on Linux Debug dt1 nor android-4-0-armv7-api11-p opt build. Assuming TRY infra.
https://hg.mozilla.org/mozilla-central/rev/6bf4d29e7272
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox45: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: