Closed
Bug 1516782
Opened 6 years ago
Closed 6 years ago
GeckoViewScrollChild should probably use Visual Viewport
Categories
(GeckoView :: General, enhancement, P2)
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.
Updated•6 years ago
|
Priority: -- → P2
Assignee | ||
Comment 1•6 years ago
|
||
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.
Assignee | ||
Comment 2•6 years ago
|
||
Marking as duplicate because the patch now both depends on parts of bug 1498812, but also blocks later parts of that bug.
Updated•6 years ago
|
Attachment #9034281 -
Attachment is obsolete: true
You need to log in
before you can comment on or make changes to this bug.
Description
•