Page MenuHomePhabricator

[Regression] Space missing between revision compare buttons and meta data
Closed, ResolvedPublic

Description

The spaces between the "diff" parenthesis and revision compare buttons is missing.

The spaces between the two revision compare buttons is also missing.

Expected

As in production today:

https://en.wikipedia.org/w/index.php?title=Main_Page&action=history

Screenshot 2021-01-28 at 03.19.19.png (448×746 px, 94 KB)

Actual

Currently on Beta Cluster and on group0 in production, such as mediawiki.org:

QA Results - Beta

ACStatusDetails
1T273148#6803941
2T273148#6803941

QA Results - Prod

ACStatusDetails
1T273148#6811215
2T273148#6811215

Event Timeline

Change 659258 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/core@master] HistoryAction: Fix margins of normalized input controls

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

Change 659258 merged by jenkins-bot:
[mediawiki/core@master] HistoryAction: Fix margins of normalized input controls

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

Does this task also cover the fact that there are two radio inputs in a row on intermediate revisions, or should I open a new task about that?

@Nirmos I think that is by design, clicking the left one will bring "up" the lower selection to it, clicking the right one will bring "down" the higher selection to it.

Does this task also cover the fact that there are two radio inputs in a row on intermediate revisions, or should I open a new task about that?

No, that has always been the case. Ever since the "compare revisions" feature was added to page histories, the unstyled page has had two radiobuttons for every row, and JavaScript is used to hide (by means of a visibility:hidden; declaration) those in the left-hand group adjacent to and above the selected button in the right-hand group, and to hide those in the right-hand group adjacent to and below the selected button in the left-hand group. This much has not changed: the recent change is merely that a margin:0; declaration has been added to those buttons, causing them to sit closer together.

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA

Test Artifact(s):

QA Steps

✅ AC1: The spaces between the "diff" parenthesis and revision compare buttons are NOT missing.

VectorMonobook
Screen Shot 2021-02-04 at 6.49.05 AM.png (701×1 px, 245 KB)
Screen Shot 2021-02-04 at 6.49.45 AM.png (701×1 px, 305 KB)

✅ AC2: The spaces between the two revision compare buttons are NOT missing.
see above

Edtadros subscribed.

Test Result - Beta

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

Test Artifact(s):

QA Steps

✅ AC1: The spaces between the "diff" parenthesis and revision compare buttons are NOT missing.

VectorMonobook
Screen Shot 2021-02-08 at 6.17.25 AM.png (900×996 px, 317 KB)
Screen Shot 2021-02-08 at 6.18.04 AM.png (900×996 px, 379 KB)

✅ AC2: The spaces between the two revision compare buttons are NOT missing.
see above