Page MenuHomePhabricator

DropdownInputWidget (WikimediaUI theme) cuts off the bottom of characters
Closed, ResolvedPublic

Description

@Stryn reported that the bottom of some characters are cut off, e.g. g and [ and ] are most visibly affected in this screenshot. He's using Firefox 59.0.2 on Windows 10 with default font (Times New Roman).

JyeuS8H.jpg (926×1 px, 60 KB)

I cannot reproduce this (on Firefox, Linux, with DejaVu Sans).

Event Timeline

When the page is not zoomed (100%)When I zoom a little (110%) it shows correctly.
stBmI9X - Imgur.png (546×713 px, 24 KB)
64jZecq - Imgur.png (592×773 px, 28 KB)

@Charlie_WMDE could reproduce it in the widget included in the AdvancedSearch extension ( Firefox Quantum 59.0.1 for Ubuntu)

line-height is explicitly set as for many other widgets. We could set it to normal, but that would result in probably a tad misaligned icons. Before doing this, I want to reproduce it myself. *Switches on an Windows 10 computer*

Volker_E triaged this task as Medium priority.May 28 2018, 10:00 PM
Volker_E triaged this task as Medium priority.

I've been able to reproduce this on Firefox/Windows 10.

Could this be fixed? It's annoying to see it every day in special:block....

Change 471902 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] WikimediaUI theme: Tame cut-off letter issue

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

Screenshot with patch provided:

image.png (586×688 px, 129 KB)

Volker_E renamed this task from OOUI\DropdownInputWidget cuts off the bottom of characters to DropdownInputWidget (WikimediaUI theme) cuts off the bottom of characters.Nov 6 2018, 2:28 AM
Volker_E moved this task from Backlog to Reviewing on the OOUI board.

Change 471902 merged by jenkins-bot:
[oojs/ui@master] WikimediaUI theme: Tame cut-off letter issue

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

Volker_E edited projects, added OOUI (OOUI-0.29.4); removed OOUI.
Volker_E claimed this task.
Volker_E removed a project: Patch-For-Review.
Volker_E removed a subscriber: gerritbot.

Change 472080 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@master] Update OOUI to v0.29.4

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

Change 472080 merged by jenkins-bot:
[mediawiki/core@master] Update OOUI to v0.29.4

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