Page MenuHomePhabricator

[web] Make button size consistent between variants
ClosedPublic

Authored by tomek on Jun 24 2022, 8:23 AM.
Tags
None
Referenced Files
F3493418: D4353.id13835.diff
Thu, Dec 19, 3:19 AM
F3493416: D4353.id13764.diff
Thu, Dec 19, 3:19 AM
F3493405: D4353.id.diff
Thu, Dec 19, 3:18 AM
F3493400: D4353.diff
Thu, Dec 19, 3:16 AM
F3491097: D4353.diff
Wed, Dec 18, 6:17 PM
Unknown Object (File)
Wed, Dec 18, 12:23 PM
Unknown Object (File)
Wed, Dec 18, 8:35 AM
Unknown Object (File)
Sun, Dec 15, 4:53 PM
Subscribers

Details

Summary

Our button has different size depending on the variant, because secondary button has a border that increases its size. The issue isn't usually noticeable, but can be detected by using the debugger. It becomes visible when secondary and primary buttons are displayed in the same line.
The solution is for button to always have a border, which is transparent when the button is primary.

Test Plan

Checked in a couple of places if it still looks ok. There are places when sizing is slightly changed, because primary button is 2 pixels bigger, but the layout is ok in every place.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek requested review of this revision.Jun 24 2022, 8:28 AM

Yup, this makes sense to me. I think the difference in border size was overlooked in D3057, but nice catch!

This revision is now accepted and ready to land.Jun 24 2022, 9:56 AM