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.