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)
Firefox for Android Graveyard
General
Tracking
(fennec1.0b2+)
RESOLVED
FIXED
fennec1.0b2
Tracking | Status | |
---|---|---|
fennec | 1.0b2+ | --- |
People
(Reporter: Gavin, Assigned: Gavin)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Updated•16 years ago
|
OS: Windows XP → All
Hardware: PC → All
Updated•16 years ago
|
Flags: blocking-fennec1.0+
Priority: -- → P2
Target Milestone: --- → Fennec A2
Assignee | ||
Comment 1•16 years ago
|
||
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).
Updated•16 years ago
|
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WORKSFORME
Assignee | ||
Comment 3•16 years ago
|
||
This is still broken (both for anchor clicks and scrollTo)...
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Comment 4•16 years ago
|
||
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 ?
Assignee | ||
Updated•16 years ago
|
Summary: content scrolling itself can break panning → content scrolling itself and anchor link clicks don't cause a redraw
Updated•16 years ago
|
Target Milestone: Fennec A2 → Fennec A3
Updated•16 years ago
|
tracking-fennec: --- → 1.0b2+
Comment 6•16 years ago
|
||
the summary's don't really match, but this is a dupe of 464984
Status: REOPENED → RESOLVED
Closed: 16 years ago → 16 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 7•15 years ago
|
||
Going to reopen this and mark a dependency.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Assignee | ||
Updated•15 years ago
|
Assignee | ||
Comment 8•15 years ago
|
||
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)
Assignee | ||
Updated•15 years ago
|
Status: REOPENED → ASSIGNED
Updated•15 years ago
|
Attachment #384002 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 9•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago → 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•