HomePhabricator
Diffusion Comm b1858ff3fc16

[web] Pop up a pin/unpin modal when the tooltip icon is clicked

Description

[web] Pop up a pin/unpin modal when the tooltip icon is clicked

Summary:
Following the designs, we need an intermediate modal when a user tries to pin / unpin a message that 1) displays the message and 2) allows the user to confirm their requested action.

Because we are rendering a Message, there were two Providers we needed to avoid the invariants.

Firstly, the useMessageTooltipReplyAction requires an inputState. I define the inputState in tooltip-utils since the target message should
have access to the context, as opposed to it being null in the modal.

Secondly, the messageListContext is required to be defined since Message renders a TextMessage.

Linear: https://linear.app/comm/issue/ENG-3446/pop-up-a-pinunpin-modal-when-the-tooltip-icon-is-clicked

Depends on D7308

Test Plan:
The video is linked below, I sync'd offline with Ted to confirm the designs

Reviewers: atul, ginsu, tomek

Reviewed By: tomek

Subscribers: ashoat

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

Details

Provenance
RohanK6 <rohan.khanderia@gmail.com>Authored on Apr 21 2023, 4:38 AM
rohanCommitted on Apr 25 2023, 5:58 PM
Reviewer
tomek
Differential Revision
D7309: [web] Pop up a pin/unpin modal when the tooltip icon is clicked
Parents
rCOMM1b1325634575: [lib/web] Add pin/unpin icon in the tooltip
Branches
Unknown
Tags
Unknown