Page MenuHomePhabricator

[web] Remove `connectDropTarget` from `ChatMessageList` with new `tooltip` changes
AbandonedPublic

Authored by abosh on Aug 1 2022, 9:29 AM.
Tags
None
Referenced Files
F3571474: D4700.diff
Sat, Dec 28, 9:21 AM
Unknown Object (File)
Fri, Dec 27, 5:49 AM
Unknown Object (File)
Mon, Dec 16, 12:18 AM
Unknown Object (File)
Mon, Dec 2, 12:09 PM
Unknown Object (File)
Nov 16 2024, 9:53 AM
Unknown Object (File)
Nov 8 2024, 3:23 PM
Unknown Object (File)
Nov 8 2024, 3:22 PM
Unknown Object (File)
Nov 8 2024, 3:19 PM
Subscribers

Details

Summary

In D4434, there were certain changes made that made D4481 broken since ChatMessageList was refactored. This diff brings ChatMessageList up to date and out of a broken state. (This is also the reason D4697 and D4698 were failing builds, since the old ChatMessageList code from D4481 was still in my stack because it wasn't landed prior to D4434.) This is a diff in a stack of diffs that will resolve the timestamp bug, related Linear issue here. The purpose of this diff is just to fix ChatMessageList since it was in a broken state for me, the rest of the tooltip diffs will come later in the stack.

Depends on D4481

Test Plan

Tested on web and everything looks as expected:

Diff Detail

Repository
rCOMM Comm
Branch
refactorchatmessagelist (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage