Page MenuHomePhabricator

[web] introduce functionality to send a reaction by clicking on the reaction pill
ClosedPublic

Authored by ginsu on Aug 14 2023, 9:44 AM.
Tags
None
Referenced Files
F3387747: D8808.id29866.diff
Fri, Nov 29, 10:57 AM
Unknown Object (File)
Tue, Nov 26, 4:55 AM
Unknown Object (File)
Mon, Nov 25, 9:11 PM
Unknown Object (File)
Sat, Nov 23, 1:00 PM
Unknown Object (File)
Fri, Nov 22, 3:06 PM
Unknown Object (File)
Fri, Nov 1, 1:25 PM
Unknown Object (File)
Oct 28 2024, 7:45 AM
Unknown Object (File)
Oct 28 2024, 7:45 AM
Subscribers

Details

Summary

To improve the user experience of the InlineEngagement component we are updating the clicking behavior for our reaction pills. Before whenever a user would click on a reaction, the MessageReactionsModal would pop up, but now if a user clicks a reaction pill they will react/unreact to that message depending on whether or not they have already reacted to that message already. To view the MessageReactionsModal the user now needs to hover on the reaction pill and click on the popup (a similar pattern with other chat apps such as discord and slack), and that functionality will be handled in subsequent diffs

Depends on D8807

Test Plan

Please watch the demo below:

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

This revision is now accepted and ready to land.Aug 16 2023, 2:15 AM