Closed Bug 1516782 Opened 6 years ago Closed 6 years ago

GeckoViewScrollChild should probably use Visual Viewport

Categories

(GeckoView :: General, enhancement, P2)

All
Android
enhancement

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1498812

People

(Reporter: JanH, Assigned: JanH)

References

Details

Attachments

(1 obsolete file)

I'm not quite sure for what use cases this was intended or whether there already are any GeckoView consumers using this API, but given that bug 1371796 mentioned pull-to-refresh as an example, we probably want to expose the user-visible scroll position here, which means the visual viewport nowadays.
Priority: -- → P2
Assignee: nobody → jh+bugzilla
Blocks: 1498812
Given the usage example of pull-to-refresh in bug 1371796, downstream consumers will probably more interested in the true visible scroll position of the user within the page, i.e. the visual viewport. Listening for *visual* viewport events will also definitively be required to get the saveAndRestoreState GeckoView test properly working once we switch Gecko's session store helper function to use the *visual* viewport scroll position.
Marking as duplicate because the patch now both depends on parts of bug 1498812, but also blocks later parts of that bug.
No longer blocks: 1498812
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
Attachment #9034281 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: