HomePhabricator
Diffusion Comm 77f4ba67d208

[web] introduce panel header component

Description

[web] introduce panel header component

Summary:
As I have been working on the redesign and have been converting our ui to utilize the new Panel component, I noticed that a lot of the headers followed a similar ui pattern. Rather than have this pattern be copied and pasted a bunch, I created this PanelHeader component that can be used to factor and abstract that logic out. The pattern I was noticing with the panel headers was that many panel headers either would just have the label, or they would have an optional add button.

Screenshot 2024-01-11 at 12.33.21 PM.png (214×1 px, 17 KB)

Screenshot 2024-01-11 at 12.33.52 PM.png (230×1 px, 20 KB)

Screenshot 2024-01-11 at 12.32.56 PM.png (332×752 px, 21 KB)

Screenshot 2024-01-11 at 12.33.30 PM.png (172×372 px, 10 KB)

Screenshot 2024-01-11 at 12.33.09 PM.png (154×1 px, 12 KB)

Test Plan: flow + confirmed that there were no regressions with the app list header

Reviewers: atul, inka

Reviewed By: atul

Subscribers: ashoat, tomek

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

Details

Provenance
ginsuAuthored on Jan 8 2024, 3:03 PM
Reviewer
atul
Differential Revision
D10610: [web] introduce panel header component
Parents
rCOMM0107bf19c4e7: [nix] Update dev-shell.nix with comment on SystemConfiguration
Branches
Unknown
Tags
Unknown