Page MenuHomePhabricator

[web] introduce ReactionPill component
ClosedPublic

Authored by ginsu on Aug 15 2023, 10:19 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 14, 9:36 PM
Unknown Object (File)
Fri, Jun 14, 4:30 AM
Unknown Object (File)
Tue, Jun 11, 8:22 PM
Unknown Object (File)
Sun, Jun 9, 11:56 AM
Unknown Object (File)
Tue, Jun 4, 12:11 AM
Unknown Object (File)
Mon, Jun 3, 12:07 PM
Unknown Object (File)
Mon, Jun 3, 10:08 AM
Unknown Object (File)
Mon, May 27, 9:56 PM
Subscribers

Details

Summary

As I was working on implementing the reaction tooltip I was having to put a ton of reaction specific code into InlineEngagement. This was getting messy really quick so I created and introduced this ReactionPill component to keep the logic organized

Depends on D8810

Test Plan

flow and no regressions with the ReactionPill (The pill looks the same and I can still react/unreact by clicking on the pill)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

remove accidental reaction-tooltip.react.js file

kamil added inline comments.
web/chat/reaction-pill.react.js
16–18 ↗(On Diff #29916)

I think we should avoid passing large objects to child component when possible

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

address feedback and rebase before landing

This revision was automatically updated to reflect the committed changes.