Page MenuHomePhabricator

Minerva icon spacing issue - Watch and View Source
Closed, ResolvedPublic2 Estimated Story PointsBUG REPORT

Description

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

What happens?:

  • There is a large gap between the lock icon and the associated "View Source" text, resulting it the icon instead being immediately next to "Watch". Then there is a large gap between "Watch" and the watchlist star.
  • The icons are not flushed correctly and are not aligned with the article content.

Screenshot 2025-08-06 at 7.26.37 PM.png (850×2 px, 1 MB)

What should have happened instead?:

The lock icon should be next to "View Source" and the watchlist star should be closer to the "Watch" text.

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):

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

Event Timeline

Esanders subscribed.

...or more commonly Watch and Edit (and Download PDF):

image.png (107×366 px, 6 KB)

There is another probably related bug in advanced mode where the icons go beyond the edge of the center column:

image.png (250×1 px, 51 KB)

It looks like some padding was removed from the large codex buttons, probably upstream. Here is how it looked before the regression:
https://patchdemo.wmcloud.org/wikis/be7e72f4e3/w/index.php?title=New_York_City&mobileaction=toggle_view_mobile

The reason for this issue in case was https://gerrit.wikimedia.org/r/c/mediawiki/skins/MinervaNeue/+/1177485/3/resources/skins.minerva.styles/buttons.less.
The selector was originally added with a mobile-only use case, where the labels are invisible and it's icon-only buttons, hence the line
<ul id="p-views" class="page-actions-menu__list minerva-icon-only-menu"> in 'PageActionsMenu.mustache'
negating the differences on tablet plus.

Issue is, on anything above 1120px the icon-only labels are set visible again by not applying invisible hack to the label

@media (max-width: calc(1119px)) {
  .cdx-button.cdx-button--icon-only span + span:last-child

The right approach would be to change the CSS class accordingly or, less bullet-proof, apply margin-end differently.

Change #1184175 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Fixes Minerva icon spacing issue on tablet/desktop

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

T403816 may be related (codex button padding issues)

Change #1184175 abandoned by Jdlrobson:

[mediawiki/skins/MinervaNeue@master] styles: Fix PageActions icon spacing issue on tablet/desktop

Reason:

Not working on this.

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

I spoke with @Sneha about the horizontal padding and she advised to leave as is since this is on Minerva and thus mostly touch screens where hover and focus states aren't as used/important. So the original patch LGTM, I'm going to restore it and move to code review.

Change #1184175 restored by LorenMora:

[mediawiki/skins/MinervaNeue@master] styles: Fix PageActions icon spacing issue on tablet/desktop

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

Change #1184175 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] styles: Fix PageActions icon spacing issue on tablet/desktop

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

Etonkovidova subscribed.

Checked in betalabs for the following cases
(1) Desktop + MinervaNeue skin
(2) Desktop + MinervaNeue skin (responsive mode disabled)
(3) Mobile + Advanced mode disabled
(4) Mobile + Advanced mode enabled

betalabs - Desktop+MinervaNeuewmf.4 (for comparison) Desktop+MinervaNeue
Screenshot 2025-12-01 at 6.20.22 PM.png (596×2 px, 133 KB)
Screenshot 2025-12-01 at 6.20.33 PM.png (534×2 px, 118 KB)

No issues found; will also check after wmf.5 deployment.

Volker_E removed a project: Patch-For-Review.

The icons flush not perfectly scalpel cut, but are harmoniously intended on mobile and don't feature the description issues any longer.

Test wiki on Patch demo by ESanders (WMF) using patch(es) linked to this task was deleted:

https://d9fc83fd11.catalyst.wmcloud.org/w/

Test wiki on Patch demo by ESanders (WMF) using patch(es) linked to this task was deleted:

https://d9fc83fd11.catalyst.wmcloud.org/w/