Page MenuHomePhabricator

Broken image in the hint to the settings link on mobile view
Closed, ResolvedPublic1 Estimated Story PointsBUG REPORT

Description

  • I am an anonymous user
  • Open the mobile view
  • Hover a page preview
  • Get into the settings using the cogwheel
  • Disable page previews and hit save
  • I get this window

Broken image.png (420×576 px, 13 KB)

Somehow the image is broken there (it seems to be very small).

Event Timeline

Suggestion: Let's do a quick git bisect and see if we find a specific patch that might be responsible for this.

Change 683519 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/Popups@master] Fix image size on footer link help screen

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

thiemowmde added a project: Page-Previews.
thiemowmde changed the subtype of this task from "Task" to "Bug Report".
thiemowmde moved this task from Sprint Backlog to Review on the WMDE-TechWish-Sprint-2021-04-28 board.
thiemowmde set the point value for this task to 1.

According to my first git bisect in the Popups codebase this issue exists since the very first patch https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Popups/+/571465/ (T236097, April 2020) that enabled Page-Previews on mobile.

Digging deeper reveals the patch https://gerrit.wikimedia.org/r/c/mediawiki/skins/MinervaNeue/+/540479 in the mobile skin (T233521, October 2019). This introduced the background-size value that's responsible for this issue.

Either way, it's not a Regression but probably like this ever since.

Fix works well for the image. There's still some misplacement with the text maybe it's equally easy to fix. See the images below.

Desktop

HelpDesktop.png (290×430 px, 14 KB)

Mobile

HelpMobile.png (313×430 px, 12 KB)

Change 683519 merged by jenkins-bot:

[mediawiki/extensions/Popups@master] Fix image size on footer link help screen

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

Change 683573 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/Popups@master] Fix image position on the settings dialog help screen

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

Change 683573 merged by jenkins-bot:

[mediawiki/extensions/Popups@master] Fix image position on the settings dialog help screen

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