HomePhabricator
Diffusion Comm 6bda99d0eafd

[web] Display the pinned messages for the thread when the banner is clicked

Description

[web] Display the pinned messages for the thread when the banner is clicked

Summary:
The pinned messages modal for the thread should display when the user clicks on the banner indicating the number of pinned messages for that given thread. Each message should display the inline engagement (reactions and replies), based on the designs. This is different from the message being displayed in the toggle pin modal, as the designs there indicated that we wanted to remove the inline engagement.

Depends on D7310

Linear: https://linear.app/comm/issue/ENG-3449/display-the-pinned-messages-for-the-thread-when-the-banner-is-clicked

There is a follow-up task to help refactor usability for the messages search screen here: https://linear.app/comm/issue/ENG-3644/refactor-file-names-details-to-make-the-thread-pin-modal-reusable-by

Test Plan:
Pin several messages, with / without inline engagement and a mix between text and media. Confirm that the media gallery looks as expected.

Reviewers: atul, ginsu, tomek, inka

Reviewed By: tomek

Subscribers: ashoat

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

Details

Provenance
RohanK6 <rohan.khanderia@gmail.com>Authored on Apr 24 2023, 1:46 PM
rohanCommitted on Apr 25 2023, 5:59 PM
Reviewer
tomek
Differential Revision
D7380: [web] Display the pinned messages for the thread when the banner is clicked
Parents
rCOMMb18a354ea61d: [web] Display the number of pinned messages in a banner across chat
Branches
Unknown
Tags
Unknown