Page MenuHomePhabricator

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

Authored by ashoat on Wed, Oct 9, 8:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 29, 7:20 AM
Unknown Object (File)
Thu, Oct 24, 2:10 PM
Unknown Object (File)
Wed, Oct 23, 9:53 PM
Unknown Object (File)
Wed, Oct 23, 3:50 AM
Unknown Object (File)
Mon, Oct 21, 4:54 AM
Unknown Object (File)
Sat, Oct 19, 5:26 PM
Unknown Object (File)
Sat, Oct 19, 12:43 AM
Unknown Object (File)
Sat, Oct 19, 12:05 AM
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