HomePhabricator
Diffusion Comm 6497c6952071

[web] Fix `See more...` button alignment regression

Tags
None
Referenced Files
F50353: Screen Shot 2022-05-04 at 11.08.13 AM.png
May 4 2022, 4:53 PM
F50355: image.png
May 4 2022, 4:53 PM
Subscribers
None

Description

[web] Fix See more... button alignment regression

Summary:
Noticed a regression in the See more... sidebar button's alignment with the sidebars. Fixed this by adjusting the padding-left.

As an aside, because the ChatThreadList component is being actively worked on, I suspect the alignment may be messed up in the future if the arrow/sidebar get bigger or smaller. Therefore, we should probably pull in the padding from some central place so we don't introduce new regressions when modifying the styling of one part of the component.

Test Plan:
Tested on Chrome/Safari and works as expected:
Before:

Screen Shot 2022-05-04 at 11.08.13 AM.png (384×810 px, 35 KB)

After:
image.png (398×732 px, 33 KB)

Reviewers: atul, benschac

Reviewed By: atul

Subscribers: benschac, ashoat, palys-swm, Adrian, karol-bisztyga

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

Details

Provenance
aboshAuthored on May 4 2022, 11:10 AM
Reviewer
atul
Differential Revision
D3917: [web] Fix `See more...` button alignment regression
Parents
rCOMM4f08459ae2cf: [native] Increase contrast between `ColorPickerModal` and background view
Branches
Unknown
Tags
Unknown