HomePhabricator
Diffusion Comm 24839908bd81

[web] Split ThreadTopBar into two components

Tags
None
Referenced Files
F383116: image.png
Mar 13 2023, 6:35 AM
File Not Attached
F383113: image.png
Mar 13 2023, 6:35 AM
File Not Attached
Subscribers
None

Description

[web] Split ThreadTopBar into two components

Summary:
Issue: https://linear.app/comm/issue/ENG-2740/add-navigation-state-info-to-the-top-bar
Navigation state info will be displayed in the Topbar component. It used to be in ChatMessageList container. Now, in ChatListContainer we will only display the name of the chat.
I split ThreadTopBar component into two: the one that will be displayed in the Tobbar, that includes the ThreadAncestors, but doesn't have the ThreadMenu, and a complementary component, that is used in the
ChatMessageListContainer. I will change the NavStateInfoBar further, accordinng to the designs, in the following diffs.
Some files were additionally moved to keep the code correctly divided into folders.

image.png (1×1 px, 139 KB)

Test Plan:
Run web app, checked that the header of ChatMessageListContainer displays only chat's: name, colour square and menu button. I also exchanged ThreadTopBar in ChatMessageListContainer for NavStateInfoBar and check that is displays correctly:

image.png (1×1 px, 144 KB)

Reviewers: michal, kuba, kamil

Reviewed By: michal

Subscribers: ashoat, tomek, atul

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

Details

Provenance
InkaSokolowskaAuthored on Mar 13 2023, 3:28 AM
Reviewer
michal
Differential Revision
D6745: [web] Split ThreadTopBar into two components
Parents
rCOMM15dad779d824: [web] Move logic for fetching ThreadInfo for possibly pending chat out of…
Branches
Unknown
Tags
Unknown