Page MenuHomePhabricator

ProcessDialog: Vertical alignment of buttons off
Closed, ResolvedPublic

Description

A pixel or two too low:

pasted_file (57×96 px, 1 KB)

pasted_file (54×84 px, 1 KB)

Event Timeline

Esanders raised the priority of this task from to High.
Esanders updated the task description. (Show Details)
Esanders added a project: OOUI.
Esanders added subscribers: Esanders, Prtksxna.

Change 240384 had a related patch set uploaded (by Jforrester):
Fix icon/indicator padding on TextInputWidget/SelectFileWidget

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

Esanders renamed this task from Vertical alignment of dialog buttons off in Chrome to Vertical alignment of buttons off in Chrome.Sep 24 2015, 10:59 AM
Esanders updated the task description. (Show Details)
Esanders set Security to None.

Above patch is not related

Jdforrester-WMF subscribed.

Looks OK to me right now – is this fixed?

Screen Shot 2016-03-01 at 17.56.07.png (117×1 px, 12 KB)

@Jdforrester-WMF No, it's not. It doesn't look like the button is too low at top, but it's takes up a bit too much height altogether and doesn't get appropriate padding-bottom (distance to the bottom). It starts at the right top point, but goes .x pixels too far down.
After looking just for a minute at it, it's another bug related to the cumbersome inabilities [[ https://phabricator.wikimedia.org/T130691 | to handle ems in general appropriately ]].

Still an issue post-0.18.0 I think.

Change 342507 had a related patch set uploaded (by VolkerE):
[oojs/ui] MediaWiki theme: Ensure vertical alignment of dialog top bar items

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

Volker_E moved this task from Next-up to Reviewing on the OOUI board.
Volker_E renamed this task from Vertical alignment of buttons off in Chrome to ProcessDialog: Vertical alignment of buttons off .Mar 13 2017, 11:35 PM

Change 342507 merged by jenkins-bot:
[oojs/ui] MediaWiki theme: Ensure vertical alignment of dialog top bar items

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