HomePhabricator
Diffusion Comm a9a0c58936cc

[native] Redesign `InlineSidebar` for reactions feature

Description

[native] Redesign InlineSidebar for reactions feature

Summary:
Introduce redesigned InlineSidebar component for native. It is now able to keep both sidebar and reactions.
Now it receives reactions as an array of strings - so it would be easy to test visually by providing an array of emojis. Probably, in the future we will keep reactions in some more complex data structure, so the types should be adjusted then.


Screenshot_Simulator_2022-08-12_115809.png (588×565 px, 33 KB)

Screenshot_Simulator_2022-08-12_115833.png (262×571 px, 26 KB)

Screenshot_Simulator_2022-08-12_120031.png (777×553 px, 481 KB)

Screenshot_Simulator_2022-08-12_120112.png (163×393 px, 12 KB)

GIF:
Screenshot_Simulator_2022-08-12_120400.gif (160×560 px, 620 KB)

Test Plan:
Tested the component look on both platforms in both color themes. Also provided smoji and tested displaying reactions and/or sidebar.
The visual issues when opening message tooltip will be fixed in next diff.

Reviewers: tomek, atul, abosh, ashoat

Reviewed By: tomek, atul, abosh, ashoat

Subscribers: ashoat, adrian

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

Details

Provenance
jacekAuthored on Aug 25 2022, 1:37 AM
Reviewer
tomek
Differential Revision
D4824: [native] Redesign `InlineSidebar` for reactions feature
Parents
rCOMMbb563918728b: [lib] Make `useInlineSidebarText` hook parameter optional
Branches
Unknown
Tags
Unknown