Page MenuHomePhabricator

Page scrolls to the very top on a small scroll up gesture
Open, Needs TriagePublic

Description

Steps to reproduce:

Observed Result:

  • Page scrolls up to the very top
  • (Sometimes the page moves to the very top autonomously while you scroll down too)

Expected Result:

  • Page scrolls up based on the exact interaction/gesture

Screenshot/Screen capture:

Device:

  • Samsung Galaxy A12 (Android 12)

Browser:

  • Chrome 99.0.4844.88
  • Firefox 145.0.2

Wiki:

  • en.wikipedia.org
  • test.wikipedia.org

Editor:

  • VisualEditor

Page:

Event Timeline

Change #1213474 had a related patch set uploaded (by Adarsh2406; author: Adarsh2406):

[mediawiki/extensions/VisualEditor@master] Fix: stable smooth scroll target and remove comments Use the root scrollable element for smooth scrolling to avoid jumps on Android. Bug: T411367 Change-Id: Iea474cf72ce4ab4770373906390518aa87e25cdb Bug: T411367 Change-Id: I952dc3668f07d15cd975e3c2b5eff04343f632e9

https://gerrit.wikimedia.org/r/1213474

Change #1213474 had a related patch set uploaded (by Adarsh2406; author: Adarsh2406):

[mediawiki/extensions/VisualEditor@master] Fix: stable smooth scroll target and remove comments Use the root scrollable element for smooth scrolling to avoid jumps on Android.

https://gerrit.wikimedia.org/r/1213474

@ppelberg Growth is looking into related scrolling issues right now (scrolling within a Revise Tone suggestion which also seems to be buggy on mobile).

When testing a random article on an iPhone, I also replicate this behavior, although sometimes it's just jumpy and I don't always end up at the top.

Is this a known issue?

@ppelberg Growth is looking into related scrolling issues right now (scrolling within a Revise Tone suggestion which also seems to be buggy on mobile).

When testing a random article on an iPhone, I also replicate this behavior, although sometimes it's just jumpy and I don't always end up at the top.

Is this a known issue?

Thank you for filing, @KStoller-WMF!

Scrolling, in general, is known to be a bit flaky/finicky. Although, this particular issue seems novel to me. We'll have a look...

It's relatively likely that we've fixed this as part of the changes we merged in T416537. It's possible that the attached patch's changes to android scrolling might still be desired, though?

ppelberg added subscribers: EAkinloose, Ryasmeen.

@Ryasmeen + @EAkinloose: could y'all please test this ticket in conjunction with T416537...we think T416537 might've fixed this issue as well, per what David shared in T411367#11624534.