Page MenuHomePhabricator

Incorporate new component library release in bridge
Closed, ResolvedPublic


  • incl base font sizes for flexible components

Event Timeline

Change 572247 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/Wikibase@master] bridge: update to new component library version

Change 572247 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] bridge: update to new component library version

Charlie_WMDE added a subscriber: Charlie_WMDE.

I found some inconsistencies with the specs:

the button should follow the specs of a Button Size M

image.png (234×1 px, 37 KB)

The button on en.beta.wp:

  • has a font size of 16px with a line height of 24px but should be 14px with a line height of 21px
  • has spacing of 6px above and below the text but should be 4 above and 5 below
  • has a button height of 38px but should be 32px (should resolve itself when the line height and margins are fixed that i mentioned in the previous bullet points)

The component library is for the radio inputs – the buttons are unrelated. The adjustments you mention are currently part of T239421 and still in review (I4a8ecaea24).

Radio inputs verified – font sizes 16px and 14px, radio button 24×24px.