They used to be larger. See pholios attached.
Description
Details
Project | Branch | Lines +/- | Subject | |
---|---|---|---|---|
oojs/ui | master | +1 -1 | MediaWiki theme: Use checkbox icon per mockups |
Related Objects
Event Timeline
Sorry to clarify, the margins portions of the patches are probably fine, but need to be broken out from the sizing changes which need to be reverted .
Indepedently, the checkbox also uses the pre-existing icon, rather than the one from mockups. Seems like there is a "tick" icon in WikiFont (or whatever the name is now), but it was discarded when we imported the icons in 06a5031fd3bf6f25f2398aff2b4999bfe224b463, as it doesn't match the mockup either (and it looked just plain ugly when I tried it right now). I'm submitting a patch for that; not touching the sizes now.
Change 200635 had a related patch set uploaded (by Bartosz Dziewoński):
MediaWiki theme: Use checkbox icon per mockups
@matmarex this task is referring to the change in the OVERALL size of the control, not the size of the graphics WITHIN the control.
Okay. In the future, I will refrain from trying to implement the parts of design changes that can be implemented immediately, if there's some detail blocking the rest.
@Jdforrester-WMF, the patch that @matmarex submitted - rGOJUe0d015e20f0d - although useful, doesn't resolve this issue.
Please also see my comment on T86003#1177354.
Hey @Jdforrester-WMF, I'm reopening this, although it's a dupe because it's not yet solved.
Here's the latest:
The bit of this task that needed fixing for Editing Q4 is fixed. Please follow-up with further comments on T86003.