HomePhabricator
Diffusion Comm 1560bbf996c7

[native] style SidebarListModal, SubchannelsModal, ThreadListModal

Tags
None
Referenced Files
F307989: Screenshot 2022-12-28 at 17.36.20.png
Dec 29 2022, 9:26 AM
File Not Attached
F307990: Screenshot 2022-12-28 at 17.36.33.png
Dec 29 2022, 9:26 AM
File Not Attached
F307991: Screenshot 2022-12-28 at 17.36.54.png
Dec 29 2022, 9:26 AM
File Not Attached
F307992: Screenshot 2022-12-28 at 17.37.58.png
Dec 29 2022, 9:26 AM
File Not Attached
F307993: Screenshot 2022-12-28 at 17.37.19.png
Dec 29 2022, 9:26 AM
File Not Attached
Subscribers
None

Description

[native] style SidebarListModal, SubchannelsModal, ThreadListModal

Summary:
Linear issues: https://linear.app/comm/issue/ENG-2547/restyle-sidebarlistmodal-and-subchannelslistmodal, https://linear.app/comm/issue/DES-20/designs-for-subchannels-modal
SidebarListModal, SubchannelsModal use ThreadListModal undeneath, and no other component uses ThreadListModal. SidebarListModal and SubchannelsModal are supposed to look consistent, and match the new designs provided in the second linked issue.

Screenshot 2022-12-28 at 17.37.19.png (1×516 px, 108 KB)

Screenshot 2022-12-28 at 17.37.58.png (1×512 px, 100 KB)

Screenshot 2022-12-28 at 17.36.54.png (1×516 px, 65 KB)

Screenshot 2022-12-28 at 17.36.33.png (1×514 px, 82 KB)

Screenshot 2022-12-28 at 17.36.20.png (1×508 px, 62 KB)

Test Plan: Run ios simulator, saw that the two modals look as expected. Chceked that scrolling works poperly.

Reviewers: kamil, tomek, michal, przemek, ashoat

Reviewed By: tomek, ashoat

Subscribers: ashoat, atul

Differential Revision: https://phab.comm.dev/D6073

Details

Provenance
InkaSokolowskaAuthored on Dec 29 2022, 9:09 AM
Reviewer
tomek
Differential Revision
D6073: [native] style SidebarListModal, SubchannelsModal, ThreadListModal
Parents
rCOMMca9417df6504: [native] Remove background from modal items
Branches
Unknown
Tags
Unknown