Closed Bug 452286 Opened 16 years ago Closed 15 years ago

content scrolling itself and anchor link clicks don't cause a redraw

Categories

(Firefox for Android Graveyard :: General, defect, P2)

defect

Tracking

(fennec1.0b2+)

RESOLVED FIXED
fennec1.0b2
Tracking Status
fennec 1.0b2+ ---

People

(Reporter: Gavin, Assigned: Gavin)

References

Details

Attachments

(1 file)

Pages calling e.g. window.scrollTo will currently mess up panning, I think. We could probably use an onscroll handler (does this work?) to intercept the scroll and turn it into a pan instead, but that might still cause trouble for pages that depend on certain elements being visible in the underlying browser.
OS: Windows XP → All
Hardware: PC → All
Flags: blocking-fennec1.0+
Priority: -- → P2
Target Milestone: --- → Fennec A2
Depends on: 461804
The messing up part was mostly fixed by bug 461804. What remains is fixing the fact that content scrolling itself won't cause us to redraw, when perhaps it should. Also need to handle anchor links clicks (though that might require a different fix).
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WORKSFORME
This is still broken (both for anchor clicks and scrollTo)...
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
is there way to listen to scroll movements ? if not, would it be interesting to implement a observer/notifier for that ? so fennec could detect when scrolls happen, and update pan control variables accordingly, as well as force painting of what needs to be re-painted ?
Summary: content scrolling itself can break panning → content scrolling itself and anchor link clicks don't cause a redraw
Target Milestone: Fennec A2 → Fennec A3
tracking-fennec: --- → 1.0b2+
the summary's don't really match, but this is a dupe of 464984
Status: REOPENED → RESOLVED
Closed: 16 years ago16 years ago
Resolution: --- → DUPLICATE
Going to reopen this and mark a dependency.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Assignee: nobody → gavin.sharp
No longer depends on: 461804
Target Milestone: B1 → B2
Attached patch patch (deleted) — Splinter Review
This is almost identical to the patch posted by Vivien in bug 470861 (and also mostly fixes that bug). It just makes us pan the widgetstack when the page scrolls itself. I filed bug 499212 for the opposite case (scrolling the document when the widgetstack is panned), since it introduces some extra complexities.
Attachment #384002 - Flags: review?(mark.finkle)
Status: REOPENED → ASSIGNED
Attachment #384002 - Flags: review?(mark.finkle) → review+
Status: ASSIGNED → RESOLVED
Closed: 16 years ago15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: