Closed Bug 461804 Opened 11 years ago Closed 11 years ago

rework event redispatching and element detection to avoid scrolling the viewport (can't click link on nytimes.com)

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
fennec1.0a2

People

(Reporter: Gavin, Assigned: Gavin)

References

Details

Attachments

(1 file, 1 obsolete file)

Once bug 461803 and bug 459604 are fixed, we should be able to make use of both of those methods.
Attached patch patch (obsolete) — Splinter Review
Makes use of the methods added in bug 459604 and bug 461803 to avoid the need to scroll the document to click links. This fixes nytimes.com, bug 461846, and most of bug 452286 (we still don't redraw when content scrolls itself, but at least it doesn't mess us up anymore).
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attached patch real patchSplinter Review
Attachment #346542 - Attachment is obsolete: true
Attachment #346543 - Flags: review?(pavlov)
Comment on attachment 346543 [details] [diff] [review]
real patch

this looks good, look forward to never scrolling again
Attachment #346543 - Flags: review?(pavlov) → review+
http://hg.mozilla.org/mobile-browser/rev/b471f5037a80
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Fennec A2
No longer blocks: 452286
You need to log in before you can comment on or make changes to this bug.