Page MenuHomePhabricator

[native] Don't create space for author name in RobotextMessageTooltipModal
ClosedPublic

Authored by ashoat on Wed, Oct 9, 8:27 PM.
Tags
None
Referenced Files
F3102039: D13680.diff
Wed, Oct 30, 10:23 AM
F3101160: D13680.id45039.diff
Wed, Oct 30, 6:19 AM
Unknown Object (File)
Mon, Oct 28, 9:25 PM
Unknown Object (File)
Sat, Oct 26, 10:20 AM
Unknown Object (File)
Fri, Oct 25, 11:50 AM
Unknown Object (File)
Fri, Oct 25, 7:42 AM
Unknown Object (File)
Thu, Oct 24, 8:45 PM
Unknown Object (File)
Thu, Oct 24, 2:10 AM
Subscribers
None

Details

Summary

The logic for navigate to RobotextMessageTooltipModal is copied from similar logic for TextMessageTooltipModal and MultimediaMessageTooltipModal.

When the viewer selects a text or multimedia message that somebody else sent, we display the author's name. However, we don't do the same for robotext.

The code being removed here would offsite the modal to give space for the author's name. Since we don't display it for robotext, I'm removing the offset here.

Depends on D13679

Test Plan

Confirm that the tooltip doesn't appear strangely higher for non-viewer-created robotext

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable