Page MenuHomePhabricator

Scrolling issues in beta with new ReferencesDrawer
Closed, ResolvedPublic


When a long reference drawer is scolled, the drawer closes. You can test this on Diana, Princess of Wales article on Beta Labs – I made the first reference really long.,_Princess_of_Wales?mobileaction=beta

Event Timeline

Jdlrobson raised the priority of this task from to Needs Triage.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a project: Readers-Web-Backlog.
Jdlrobson moved this task to Incoming on the Readers-Web-Backlog board.
Jdlrobson added a subscriber: Jdlrobson.

On the example I can scroll on desktop so I assume this is only limited to touch devices?

gerritbot added a subscriber: gerritbot.

Change 178338 had a related patch set uploaded (by Jdlrobson):
Scrollable reference drawers


@rmoen Could you test this situation against your current patchset in Gerrit and make sure it is addressed?

This has been a primary thought of my patch. And I have tested it on IOS 8 simulator and chrome. I will test further.

@Jdlrobson Correct. In desktop, if you focus the main body you can scroll the page. In Mobile, touches outside the drawer and scrolling beyond the drawer prevents body from scrolling.

Ping @rmoen let's either get this bug fix wrapped up today or abandon the patch and free it for a volunteer.

Change 178338 abandoned by Jdlrobson:
Scrollable reference drawer beta behavior

Abandoning due to lack of activity. Please restore and repost a new patch if you want to continue to work on this.

Change 194716 had a related patch set uploaded (by Bmansurov):
Beta: Don't close ReferencesDrawer on scroll

bmansurov claimed this task.

Change 194716 merged by jenkins-bot:
Beta: Don't close ReferencesDrawer on scroll