Page MenuHomePhabricator

[wmf.12-minor] monobook Special:Notification - drop-down options for the cog icon should be aligned with the icons
Closed, ResolvedPublic

Description

In Vector skin, the icons for the cog icon options are aligned:

Screen Shot 2018-01-02 at 3.39.19 PM.png (389×1 px, 80 KB)

In monobook they are on different lines:

Screen Shot 2018-01-02 at 3.39.37 PM.png (326×494 px, 25 KB)

Event Timeline

Etonkovidova renamed this task from [wmf.12-minor] monobook Special:Notification - drop-down options for the cog icon should be aligned with the icos to [wmf.12-minor] monobook Special:Notification - drop-down options for the cog icon should be aligned with the icons.Jan 2 2018, 11:52 PM
Mooeypoo added subscribers: Volker_E, Mooeypoo.

@Volker_E this seems to be an issue in OOUI Apex.

There's a rule for Apex:

.oo-ui-widget.oo-ui-iconElement .oo-ui-iconElement-icon,
.oo-ui-widget.oo-ui-iconElement > * > .oo-ui-iconElement-icon,
.oo-ui-widget.oo-ui-iconElement .oo-ui-iconElement .oo-ui-iconElement-icon,
.oo-ui-widget.oo-ui-indicatorElement > .oo-ui-indicatorElement-indicator,
.oo-ui-widget.oo-ui-indicatorElement > * > .oo-ui-indicatorElement-indicator,
.oo-ui-widget.oo-ui-indicatorElement .oo-ui-indicatorElement .oo-ui-indicatorElement-indicator {
   display: block;
}

Beyond the fact that it's super specific and had to be overridden with "!important", we had to make sure the icon is set to display: inline-block; for the icon to appear properly inside the button, otherwise the label was pushed down.

How is this not causing issues in other buttons with icons?

Change 404585 had a related patch set uploaded (by Mooeypoo; owner: Mooeypoo):
[mediawiki/extensions/Echo@master] Fix help menu icon placement for monobook

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

Change 404585 merged by jenkins-bot:
[mediawiki/extensions/Echo@master] Fix help menu icon placement for monobook

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