HomePhabricator
Diffusion Comm 871da6be31fa

[web] Use `flex-wrap` to lay out `ColorSelectorButton`s in `ColorSelector`

Tags
None
Referenced Files
F37266: 13bb.png
Apr 12 2022, 12:00 PM
Subscribers

Description

[web] Use flex-wrap to lay out ColorSelectorButtons in ColorSelector

Summary:
Addresses feedback from @palys-swm and @def-au1t in D3662. It turned out to be way easier than I expected (if I did it right...)


Depends on D3702

Test Plan:
Layout looks correct at different zoom levels:

13bb.png (1×692 px, 89 KB)

(the background colors are just to help visualize)

Reviewers: def-au1t, palys-swm

Reviewed By: palys-swm

Subscribers: ashoat, Adrian, karol-bisztyga, benschac, yayabosh, palys-swm, def-au1t

Differential Revision: https://phabricator.ashoat.com/D3703

Details

Provenance
atulAuthored on Apr 11 2022, 1:33 PM
Reviewer
tomek
Differential Revision
D3703: [web] Use `flex-wrap` to lay out `ColorSelectorButton`s in `ColorSelector`
Parents
rCOMM97fa0a4fb3d6: [web] Pull colors in `ColorSelector` from `selectedThreadColors`
Branches
Unknown
Tags
Unknown