Page MenuHomePhabricator

In iOS Safari, after opening the hamburger menu, clicking on the greyed-out article doesn't close the hamburger menu
Closed, ResolvedPublic

Description

Tested in local master and English Wikipedia in both iOS 8 and 6.

Event Timeline

kaldari raised the priority of this task from to High.
kaldari updated the task description. (Show Details)
kaldari added a subscriber: kaldari.
phuedx set Security to None.
phuedx added a subscriber: phuedx.

Moved this over to Doing prematurely.

The .transparent-shield element doesn't seem to be bubbling click events.

Change 208953 had a related patch set uploaded (by Phuedx):
Revert "Append drawers to body so they can work in other skins"

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

According to git bisect this regression was caused by 203242.

@Jdlrobson: I'm unsure of blanket reverting the change. What do you think?

There's an ominous remark about changing #mw-mf-viewport to body in Overlay.js.

Change 209047 had a related patch set uploaded (by Jdlrobson):
Allow ios users to close the main menu

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

Change 208953 abandoned by Jdlrobson:
Revert "Append drawers to body so they can work in other skins"

Reason:
See https://gerrit.wikimedia.org/r/#/c/209047/

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

Change 209047 merged by jenkins-bot:
Allow ios users to close the main menu

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

This is a symptom of T98200.

I've tested the main menu in the iOS simulator locally.

Change 209322 had a related patch set uploaded (by Robmoen):
Allow ios users to close the main menu

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

Change 209323 had a related patch set uploaded (by Robmoen):
Allow ios users to close the main menu

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

Change 209322 abandoned by Robmoen:
Allow ios users to close the main menu

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

Change 209323 merged by jenkins-bot:
Allow ios users to close the main menu

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