Page MenuHomePhabricator

[native] removed tooltip location logic
AbandonedPublic

Authored by ginsu on Oct 20 2022, 3:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 25, 10:43 PM
Unknown Object (File)
Fri, Oct 25, 10:43 PM
Unknown Object (File)
Fri, Oct 25, 10:42 PM
Unknown Object (File)
Fri, Oct 25, 10:37 PM
Unknown Object (File)
Sep 26 2024, 10:13 PM
Unknown Object (File)
Sep 18 2024, 8:41 PM
Unknown Object (File)
Sep 9 2024, 7:51 PM
Unknown Object (File)
Sep 9 2024, 7:51 PM

Details

Reviewers
atul
rohan
ashoat
Summary

Will not land this diff until the next diff in the stack is accepted

removed the tooltip location logic since eventually, the tooltip toolbar will always be at the bottom of the chat window


Depends on D5451
Linear Task: ENG-2037

Test Plan

Please review the video demo to see that the tooltip component is now always going below the message:

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: atul, rohan.
ginsu edited the summary of this revision. (Show Details)
ginsu edited the summary of this revision. (Show Details)
ginsu requested review of this revision.Oct 20 2022, 3:20 PM
native/navigation/tooltip.react.js
315 ↗(On Diff #17795)

I wonder if it's worth keeping this feature in Tooltip just in case we want to use it in the future. @atul, what do you think?

@atul, what do you think?

I don't think it hurts to keep this positioning logic around. It might be helpful if we use Tooltip for the row of emoji reactions?

It looks like we have designs in Figma for the row of emoji to appear both above and below the message bubble:

aaac82.png (1×1 px, 178 KB)

ashoat requested changes to this revision.Oct 24 2022, 7:59 AM

Cool, agree

This revision now requires changes to proceed.Oct 24 2022, 7:59 AM

planning changes for addressing feedback

after speaking to @atul we came up with a better strategy of instead of removing the location logic, adding a fixed option to the location logic that fixes the toolbar to the bottom of the chat window