Page MenuHomePhabricator

[native] Remove TooltipInlineEngagement from RobotextMessageTooltipButton
ClosedPublic

Authored by ashoat on Wed, Oct 9, 8:24 PM.
Tags
None
Referenced Files
F3102034: D13676.diff
Wed, Oct 30, 10:21 AM
Unknown Object (File)
Wed, Oct 23, 4:40 AM
Unknown Object (File)
Tue, Oct 22, 10:06 PM
Unknown Object (File)
Sun, Oct 20, 2:23 PM
Unknown Object (File)
Fri, Oct 18, 8:47 PM
Unknown Object (File)
Fri, Oct 18, 5:33 PM
Unknown Object (File)
Fri, Oct 18, 1:46 PM
Unknown Object (File)
Thu, Oct 17, 11:47 PM
Subscribers
None

Details

Summary

This was introduced first in D4825 to solve a problem having to do with InlineEngagement appearing over the message.

But for robotext messages, we don't overlap the display of InlineEngagement with the message. As such, this problem doesn't exist for robotext messages, and doesn't need to be solved.

Depends on D13675

Test Plan

Remove TooltipInlineEngagement from text messages and multimedia messages. Observe that the issue described in D4825 appears now. Do the same for robotext messages, and confirm that the issue does not appear.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage