Page MenuHomePhabricator

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

Authored by ashoat on Oct 9 2024, 8:27 PM.
Tags
None
Referenced Files
F3512203: D13680.diff
Sat, Dec 21, 6:46 PM
Unknown Object (File)
Fri, Nov 29, 2:25 AM
Unknown Object (File)
Thu, Nov 28, 11:05 PM
Unknown Object (File)
Thu, Nov 28, 7:12 PM
Unknown Object (File)
Nov 22 2024, 8:34 AM
Unknown Object (File)
Nov 22 2024, 7:55 AM
Unknown Object (File)
Nov 22 2024, 7:45 AM
Unknown Object (File)
Nov 22 2024, 1:56 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