Example URL: https://en.m.wikipedia.org/wiki/TES#/editor/0
Description
Details
Related Objects
- Mentioned In
- T164237: Icon-only buttons in VE mobile context should provide a touch area consistent with the rest of the mobile interface (e.g. toolbar)
- Mentioned Here
- rGVEDfbeb0db2c2ae: Merge "MobileContext: fix button spacing"
rGOJU99361c772c65: MediaWiki theme: Unify `padding` on ButtonElement
rGOJUd22d23311b83: PopupToolGroup: Mixin flaggable
Event Timeline
Might be a side-effect of my changes to buttongroups in toolbars (d22d2331), somehow.
That said, is there an image somewhere of what this is supposed to look like?
I think it's actually maybe Volker! There were several commits in the last release cycle or two which involved refactoring padding on ButtonElement, particularly 99361c772c which had some impact on non-labelElement buttons. It's enough of a pain to force in a different version of OOui that I haven't tested this, but it looks more plausible.
Anyway, working on a fix now.
Change 350962 had a related patch set uploaded (by DLynch; owner: DLynch):
[VisualEditor/VisualEditor@master] MobileContext: fix button spacing
Yup, I haven't touched toolbar tools (yet), but as there are probably a few ButtonElements scattered here and there my changes might have been part of the cause. Going to help with reviewing…
My patch applies some custom padding to non-labelWidget buttons in the header of a few places in VE. I'm not sure if this would be better placed up in OOui itself; I can move it into a patch for that, if people think that's better.
From first reaction, I'd be with your comment – being inclined to have it centralized in OOUI as we've got a collection of toolbar styles and I don't see why we wouldn't want to have this generalized (without looking in all possible implementations).
Change 351341 had a related patch set uploaded (by DLynch; owner: DLynch):
[oojs/ui@master] ProcessDialog: safe action buttons were misaligned
Change 351353 had a related patch set uploaded (by Esanders; owner: Esanders):
[oojs/ui@master] Fix padding for frameless buttons in ProcessDialogs
Change 351341 abandoned by DLynch:
ProcessDialog: safe action buttons were misaligned
Reason:
Ed's patch
Change 351353 merged by jenkins-bot:
[oojs/ui@master] MediaWiki theme: Fix padding for frameless buttons in ProcessDialogs
Change 350962 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] MobileContext: fix button spacing
Change 352713 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (fbeb0db2c)
Change 352713 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (fbeb0db2c)