HomePhabricator
Diffusion Comm 2a82d05adb5e

[web] introduce community header

Description

[web] introduce community header

Summary:
This diff puts all the community header components I built in previous diffs and introduces it as the community header component. Just also want to note that this diff also includes some "hacky" logic I introduced to test the two main states of this header. This logic will be removed once I start working on improving the navigation state when navigating communites:

  1. When home/all communities is selected (there should be no community action buttons and the header should say "All communities")

Screenshot 2023-12-29 at 3.36.27 PM.png (1×1 px, 225 KB)

  1. When a community is selected (the name/avatar of the community should be in the header and the community action buttons should be rendered)

Screenshot 2023-12-29 at 3.36.20 PM.png (1×1 px, 320 KB)

Linear task: https://linear.app/comm/issue/ENG-5965/top-bar-when-a-community-is-selected and https://linear.app/comm/issue/ENG-5964/top-bar-when-inboxhome-is-selected

Depends on D10498

Test Plan:
Please see the screenshots below

When home/all communities is selected:

Screenshot 2023-12-29 at 3.39.42 PM.png (1×3 px, 809 KB)

When a community is selected:

Screenshot 2023-12-29 at 3.39.37 PM.png (1×3 px, 817 KB)

Reviewers: atul, kamil, rohan

Reviewed By: atul

Subscribers: varun, ashoat, tomek

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

Details

Provenance
ginsuAuthored on Jan 3 2024, 11:52 AM
Reviewer
atul
Differential Revision
D10499: [web] introduce community header
Parents
rCOMM912ce281d9f3: Enable e2e Windows notifications to public users
Branches
Unknown
Tags
Unknown