Redirect clicks to clickable elements nearby (smart link tapping)

RESOLVED FIXED

Status

()

Firefox for Android
General
P2
normal
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: wesj, Assigned: wesj)

Tracking

unspecified
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

(Whiteboard: [QA+])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
Created attachment 568801 [details]
WIP Patch

In old Fennec, we would look for elements near to where the user "clicked" for clickable elements. It'd be nice to do this in our new world.

Previously we had a shield that would absorb clicks, and we then redirected those clicks to the child process. In our new world the clicks happen directly on the browser. I'm playing with checking if the user actually clicked on a clickable element. If so we just let the normal world take its course. If the user only clicked close to a clickable element we fire a second "new" click on the actual clickable guy. I'm a bit worried about that firing multiple clicks and need to play with it some. Happy to hear if there are any good/better ideas about how to do this. 

In future plans, we were hoping to zoom into the area where there were multiple clickable elements.
Duplicate of this bug: 696325
Summary: Redirect clicks to clickable elements nearby → Redirect clicks to clickable elements nearby (smart link tapping)

Updated

6 years ago
Whiteboard: [QA+]

Updated

6 years ago
Assignee: nobody → wjohnston
Priority: -- → P2
(Assignee)

Comment 2

6 years ago
Created attachment 570356 [details] [diff] [review]
Patch v1

This is a lot of code, all of it copied (almost) exactly from content.js. There were numerous references to Util.js in there that I removed. I also removed a mousemove we usually fire there because it scares me to fire mousemoves with our new UI.

A bit hard to test too. I think my fingers have adjusted to tapping on stuff correctly. But some logging along with some light browsing shows this getting hit and finding the right things.
Attachment #568801 - Attachment is obsolete: true
Attachment #570356 - Flags: review?(mbrubeck)
Comment on attachment 570356 [details] [diff] [review]
Patch v1

We should try to get some of our browser-chrome tests running on birch so we can test this too.
Attachment #570356 - Flags: review?(mbrubeck) → review+
(Assignee)

Comment 4

6 years ago
http://hg.mozilla.org/projects/birch/rev/8447eaf8f0f8
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
I came across this, this May, and it seemed to be nice. Probably the 'zoom' thing can have this behavior. http://vis.berkeley.edu/papers/fingerglass/
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
You need to log in before you can comment on or make changes to this bug.