Page MenuHomePhabricator

[wmf.22 - mobile] Impact module overlay misses back navigation arrow button
Closed, ResolvedPublic2 Estimated Story PointsBUG REPORT

Description

Steps to replicate the issue:

  • On mobile, go Special:Homepage and click on the Impact module
  • The overlay won't display the back navigation arrow

Screen Shot 2023-08-18 at 9.13.58 AM.png (1×872 px, 146 KB)

What should have happened instead?:
The navigation back arrow button should be present (the screenshot below is from https://phabricator.wikimedia.org/T340199#8959486)

Screenshot 2023-06-23 at 18.31.31.png (1×800 px, 156 KB)

Acceptance Criteria:
  • Fix
  • Add a test!

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Etonkovidova renamed this task from [wmf.22 - mobile] Impact module misses a back arrow navigation icon to [wmf.22 - mobile] Impact module overlay misses back navigation arrow button.Aug 18 2023, 6:49 AM
KStoller-WMF moved this task from Sprint 0 (Growth Team) to Backlog on the Growth-Team board.
KStoller-WMF set the point value for this task to 2.
KStoller-WMF updated the task description. (Show Details)

Change 971116 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/GrowthExperiments@master] icons: Do not use :before for mw-ui-icon

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

Urbanecm_WMF subscribed.

This doesn't appear to be a Impact module specific task. Uploaded a patch that fixes the issue.

Change 971116 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] icons: Do not use :before for mw-ui-icon

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

Fixed on beta:

Screen Shot 2023-11-02 at 11.46.50 AM.png (1×800 px, 166 KB)

Also checked in testwiki wmf.4 - the fix is in place.

Change 979431 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/extensions/GrowthExperiments@master] Fixes overlay exit button and deprecation warnings

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