Page MenuHomePhabricator

[Regression] ICONS "Close" text appears below close icon when loading editor overlay
Closed, ResolvedPublicBUG REPORT

Description

Steps to reproduce

  1. Go to an editable article and click the edit icon
  2. Editing overlay loads. Notice that "Close" text shows below close icon and icon is not centered

Screen Shot 2019-09-12 at 5.31.03 PM.png (760×2 px, 430 KB)

Expected results

  • "Close" text does not appear and icon is centered in header

Actual results

  • "Close" text appears and icon is not centered

Check any additional observations

QA Results

ACStatusDetails
1T232798#5506616

Event Timeline

nray renamed this task from [Regression] to [Regression] "Close" text appears below close icon when loading editor overlay.Sep 12 2019, 11:39 PM
nray updated the task description. (Show Details)

Looks like Jon and I made a ticket for this at the same time:

https://phabricator.wikimedia.org/T232799

Jdlrobson renamed this task from [Regression] "Close" text appears below close icon when loading editor overlay to [Regression] ICONS "Close" text appears below close icon when loading editor overlay.Sep 13 2019, 12:00 AM
Jdlrobson triaged this task as High priority.

Change 536386 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/MobileFrontend@master] mobile.init/fakeToolbar: Update styles for icon changes

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

Change 536386 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] mobile.init/fakeToolbar: Update styles for icon changes

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

There is still an issue with this element, however the issue described in the task has been fixed so I suggest we close this and I will open a new task to address the gap at the bottom of the header, which may be related to T215426, and the fact that the "Loading" element is not vertically centered.

image.png (669×401 px, 107 KB)

Edtadros subscribed.

Test Result

Status: ✅ PASS
OS: macOS Mojave
Browser: Chrome
Device: MBP
Emulated Device: iPhoneX, Galaxy S5, iPad Pro, Google Pixel 2, Galaxy Note 3

Test Artifact(s):
✅ AC1 - Login button is centered in container.

T232798.png (2×1 px, 129 KB)