premii flipboard demo triggers pull to refresh |
||||
Issue descriptionWhat steps will reproduce the problem? 1. Go to https://premii.com/play/flipboard-style-news/ 2. Scroll down a few cards 3. Scroll up a few cards On stable android this works well. On canary android, this triggers pull to refresh before you get to the top of the page. Is this one of the scroll blocking interventions?
,
Sep 1 2016
No I don't think this is the root listener intervention. The touch listeners aren't on a root element.
,
Sep 1 2016
My best guess is that this is non-blocking event listeners during fling? And that is causing overscroll? I can try tomorrow. cc'ing lanwei@
,
Sep 6 2016
,
Sep 7 2016
Dave and I tried on different Android devices with PassiveEventListenersDueToFling enabled, neither of us can reproduce the problem. Could you please explain in more details how you reproduce the pull to refresh, and did you see it happen on other pages, thank you?
,
Sep 7 2016
This is a 50/50 Finch trial. So likely we are unlikely in the control group.
,
Sep 8 2016
I tested even when I made all the touch starts passive, I still did not see any pull to refresh. The page calls PreventDefault on all the touch moves, so there is no fling gesture. I guess this is not our touch event intervention.
,
Sep 8 2016
Dave's root scroller intervention will also make touchmove listeners passive - it could be that. p2r should trigger only if the document has overflow and is at scrollTop=0 (see https://docs.google.com/document/d/12Ay4s3NWake8Qd6xQeGiYimGJ_gCe0UMDZKwP9Ni4m8/edit).
,
Sep 8 2016
I thought all the handlers were added to a div. And not the body, document or window. So it wouldn't be the root scroller intervention at all.
,
Apr 24 2017
This can no longer be reproduced with Chrome M58. Closing stale issue. |
||||
►
Sign in to add a comment |
||||
Comment 1 by tdres...@chromium.org
, Sep 1 2016Status: Assigned (was: Untriaged)