HomePhabricator
Diffusion Comm fbf257c255a8

[web] introduce secondaryHeaderButton prop to Modal component

Description

[web] introduce secondaryHeaderButton prop to Modal component

Summary:
As a preliminary step to introducing the user profile menu component, we need to update the modal component. This diff makes two changes to the Modal component API

  1. Made the name prop optional
  2. introduce secondaryHeaderButton

Linear task: https://linear.app/comm/issue/ENG-5190/update-modal-component

Depends on D9447

Test Plan:
Please see the screenshot where I pass in a UserProfileMenu component (will be introduced in a subsequent diff) as the secondaryHeaderButton

Screenshot 2023-10-12 at 3.56.54 AM.png (1×3 px, 938 KB)

Reviewers: atul, inka

Reviewed By: atul

Subscribers: ashoat, tomek, wyilio

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

Details

Provenance
ginsuAuthored on Oct 11 2023, 10:22 PM
Reviewer
atul
Differential Revision
D9455: [web] introduce secondaryHeaderButton prop to Modal component
Parents
rCOMM752ab9d7c9bc: [web] implement RelationshipPromptButtons into user profiles
Branches
Unknown
Tags
Unknown