Page MenuHomePhabricator

Homepage and help panel: ask question dialog should be aware of back button
Closed, ResolvedPublic

Description

From @Etonkovidova in T222833

There is a strange behavior for back button when the back button is clicked from Mentorship/Help popup.
@kostajh - need your opinion on

  • whether the present behavior of back button is expected (and acceptable)
  • could it be a regression of some sort?
  1. Click 'Your mentor' module, then click 'Ask your mentor a question' (the same behavior for the "Get help with editing" module).
  2. A popup form will be present - click on the Back button - instead of going back, no navigation action happens. Instead the forward button becomes active but clicking on it does not provide any navigation either.

The screen recording shows no navigation actions when back/forward buttons are clicked.

  1. If the back button clicked one more time - it goes back to the side menu that sometimes occupies the whole screen:

IMG_7739.PNG (1×640 px, 53 KB)

It's worth noting that the same problem exists for the help panel -- try editing a page on mobile, opening the help panel, then tapping back. (See also T215430)

Event Timeline

MMiller_WMF renamed this task from Help panel and ask question dialog should be aware of back button to Homepage and help panel: ask question dialog should be aware of back button.Jun 18 2019, 6:46 PM
MMiller_WMF added subscribers: SBisson, Catrope, Cntlsn and 5 others.

@kostajh -- I just tried this out, and I understand it now. I do think we should have it high on the list of things to address, but it's not a blocker for the mobile release. I'm adding it to the "Desired" list in the MVP task.

Change 517917 had a related patch set uploaded (by Kosta Harlan; owner: Kosta Harlan):
[mediawiki/extensions/GrowthExperiments@master] (wip) Homepage: Open ask question dialog in route

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

Change 517917 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Homepage: Open ask question dialog in route

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

Checked in betalabs - works as expected. I'll check it also after deployment.