Page MenuHomePhabricator

Special:MergeHistory: No border to the textboxes or buttons when using the Minerva Skin
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Go to preferences and switch to the Minerva skin
  • Go to the Special:MergeHistory page

What happens?:
Text box lines for Source Page, Destination Page names, and Show Mergeable Edits are missing the text box outlines

What should have happened instead?:
Text box lines should appear for Source Page, Destination Page names, and Show Mergeable Edits, just like the other skins

Other information (browser name/version, screenshots, etc.):

Minerva- Missing text box outlines (ignore the arrows in the History section, just talking about the text box above)

T330964_IPMasking_ShowIP_NotItalicised_MinervaHistory.png (758×1 px, 155 KB)

Vector sample Having text box outlines (ignore the arrows in the History section, just talking about the text box above)

T330964_IPMasking_ShowIP_NotItalicised_DeleteMerge.png (655×3 px, 202 KB)

Event Timeline

Converting Special:MergeHistory to use OOUI should address the problem at heart here. That task is T134675.

Re-produceable on the English Wikipedia:

image.png (499×1 px, 77 KB)

Dreamy_Jazz renamed this task from Special:MergeHistory- No text box lines under the Minerva Skin to Special:MergeHistory: No border to the textboxes when using the Minerva Skin.May 11 2023, 5:54 PM
Dreamy_Jazz attached a referenced file: F36992523: image.png. (Show Details)
Dreamy_Jazz added a project: MinervaNeue.
Dreamy_Jazz updated the task description. (Show Details)

minerva has a reset.less which removes the borders of every html tag and that makes it looks like it looks on the screenshot

div, span,
h1, h2, h3, h4, h5, h6, p, blockquote, pre,
a, abbr, acronym, address, big, cite, code,
del, ins, em, img,
small, strike, strong, sub, sup, tt,
b, u, i, center,
dl, dt, dd, ol, ul, li,
fieldset, form, label, legend,
input, button, select,
audio, video {
	margin: 0;
	padding: 0;
	border: 0;
	font: inherit;
	font-size: 100%;
	vertical-align: baseline;
	background: none;
}

There are border lines on OOUI styled input widgets, but that is not the case here (T134675)

Dreamy_Jazz renamed this task from Special:MergeHistory: No border to the textboxes when using the Minerva Skin to Special:MergeHistory: No border to the textboxes or buttons when using the Minerva Skin.May 11 2023, 5:58 PM

Change 921334 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/core@master] Use HTMLForm and OOUI in Special:MergeHistory

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

Change 921334 merged by jenkins-bot:

[mediawiki/core@master] Use HTMLForm and OOUI in Special:MergeHistory

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