Page MenuHomePhabricator

pin icon looks to small since OOjs UI v0.22 got taller buttons
Closed, ResolvedPublic


OOjs UI v0.22 increased padding on buttons so they are easier to hit especially on mobile devices. This also affects the top bar of the RevisionSlider and the pin button now looks a bit lost. See screenshot.

RevisionSlider OOjs UI 0.22.png (424×1 px, 73 KB)

We could:
a) increase the size of the pin
b) reduce the padding of the bar

We should definitely do something before deployment train starts next Tuesday.

Event Timeline

OK, lets look at this together in the browser, I probably make wrong assumptions about hit area and technical possibilities if I use the image (which is very useful for seeing the problem!) :-)

Short 1on1 feedback with Jan: We will adjust the pin size to go with the taller bar.

Change 357365 had a related patch set uploaded (by WMDE-Fisch; owner: WMDE-Fisch):
[mediawiki/extensions/RevisionSlider@master] Adjust pin layout to new OOjs UI defaults

Change 357365 merged by jenkins-bot:
[mediawiki/extensions/RevisionSlider@master] Adjust pin layout to new OOjs UI defaults