HomePhabricator
Diffusion Comm 2747a10bcae8

[native/web] change title of MessageReactionsModal to All reactions

Description

[native/web] change title of MessageReactionsModal to All reactions

Summary:
I am wrapping up the inline engagement redesign work right now, but didn't have the chance to work on redesigning the MessageReactionsModal. With the new inline engagement going to our old MessageReactionsModal felt a little jarring since the modal provides all the reactions to you and not just the reaction specific ones.

To make it a little less jarring I changed the title from "Reactions" to "All reactions" and I am hoping that this will make it more clear to the user that we are not showing the list of users for that specific reactions.

https://linear.app/comm/issue/ENG-3787/implement-a-redesign-for-message-reactions-modal is the task for this, and I am hoping to get to it whenever I have a free cycle

Test Plan:
Please see screenshots

native:

Screenshot 2023-08-17 at 10.12.38 PM.png (1×1 px, 678 KB)

web:

Screenshot 2023-08-17 at 10.12.33 PM.png (2×3 px, 1018 KB)

Reviewers: atul, kamil

Reviewed By: kamil

Subscribers: ashoat, tomek

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

Details

Provenance
ginsuAuthored on Aug 17 2023, 7:09 PM
Reviewer
kamil
Differential Revision
D8861: [native/web] change title of MessageReactionsModal to All reactions
Parents
rCOMMe2a2434b7642: [web] introduce getTooltipScreenOverflowRightCorrection function
Branches
Unknown
Tags
Unknown