Page MenuHomePhabricator

[Regression] Double border on pages with empty toolbar
Closed, ResolvedPublic3 Estimated Story Points

Description

https://en.wikipedia.org/wiki/Special:Preferences
https://en.wikipedia.org/wiki/Main_Page

The toolbar on both pages despite empty has a border.

This appears to be because of
.mw-article-toolbar-container having border-bottom 1px solid #C8CCD1
.vector-page-titlebar also has a border so these become combined on special pages

Screen Shot 2022-09-15 at 3.57.56 PM.png (502×2 px, 150 KB)

Screen Shot 2022-09-15 at 3.58.01 PM.png (354×2 px, 83 KB)

QA Results - Prod

ACStatusDetails
1T317930#8262902

Event Timeline

@Jdlrobson Likely introduced by https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/817306 (T313947). The border for the page titlebar was previously applied to the h1, and after that patch it was moved to the container.

Reverting might make sense for a quick fix, or if possible moving the "display: none" style from the h1 to .vector-page-titlebar

We can revert if we have no solution by next train deployment branch cut.

I think there's actually two issues here, one is the border on the main page (which is related to T313947), and the second is the border on Special pages when the page toolbar is empty. This second issue was preexisting, and due to this selector being removed at some point.

@Jdlrobson What do you think of something like this https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/833078 to fix the first case of the main page?

Change 833078 had a related patch set uploaded (by Bernard Wang; author: Bernard Wang):

[mediawiki/skins/Vector@master] POC: Fix border on main page when title is hidden

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

Change 833078 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Fix border when title is hidden and when toolbar is empty

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

Edtadros subscribed.

@Jdlrobson I want to be certain what I'm validating. I assume the line under preferences below (copied from description) should appear sharper.

Screen Shot 2022-09-15 at 3.58.01 PM.png (354×2 px, 83 KB)

But I'm not sure what I'm validating here

Screen Shot 2022-09-15 at 3.57.56 PM.png (502×2 px, 150 KB)

@Edtadros both of those are incorrect. Check the beta cluster versions of those pages e.g. https://en.wikipedia.beta.wmflabs.org/wiki/Special:Preferences to see what they should look like:

Screen Shot 2022-09-21 at 3.57.36 PM.png (546×2 px, 146 KB)

Test Result - Prod

Status: ✅ PASS
Environment: enwiki
OS: macOS Monterey
Browser: Chrome
Device: MBP
Emulated Device:NA

Test Artifact(s):

QA Steps

✅ AC1: Double border does not appear on pages with empty toolbar

Screen Shot 2022-09-26 at 6.16.08 PM.png (211×1 px, 34 KB)

Screen Shot 2022-09-26 at 6.15.53 PM.png (226×1 px, 39 KB)

Looks good, resolving