Page MenuHomePhabricator

[native] Fix Timestamp position when focusing RobotextMessage with InlineEngagement
ClosedPublic

Authored by ashoat on Oct 9 2024, 8:28 PM.
Tags
None
Referenced Files
F3382728: D13681.diff
Thu, Nov 28, 11:34 AM
Unknown Object (File)
Mon, Nov 25, 12:04 AM
Unknown Object (File)
Sun, Nov 24, 9:50 PM
Unknown Object (File)
Mon, Nov 18, 4:55 PM
Unknown Object (File)
Mon, Nov 11, 12:22 PM
Unknown Object (File)
Mon, Nov 11, 5:37 AM
Unknown Object (File)
Sun, Nov 10, 3:11 PM
Unknown Object (File)
Sun, Nov 10, 2:06 PM
Subscribers
None

Details

Summary

We shouldn't include the InlineEngagement in the View that gets measured. By removing this, I fixed a visual glitch where the Timestamp for a robotext message would jump higher on the screen when the robotext was selected.

Depends on D13680

Test Plan
  1. I reacted to a robotext where the timestamp was already showing. Then I pressed it, and made sure there were no visual glitches or logs about incorrect height
  2. Same as above, except where the timestamp wasn't showing

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable