HomePhabricator
Diffusion Comm 6131061cecca

[native] introduced more action to tooltip actions

Description

[native] introduced more action to tooltip actions

Summary:
introduced more action to tooltip actions. The more action will only render if a fixed tooltip has more than 3 different actions. This is why only the non-viewer text message has this option. I am aware that some animations are a bit glitchy and there should only be a separator between the more button and the rest of the buttons, but I felt that addressing that was outside the scope of this diff.

Link to @expo/react-native-action-sheet repo

Here are the figma screenshots to reference the designs:

Screen Shot 2022-11-02 at 4.36.42 PM.png (1×476 px, 68 KB)

Screen Shot 2022-11-02 at 4.36.50 PM.png (1×968 px, 155 KB)


Depends on D5523 and D5497
Linear Task: ENG-2037

Test Plan:
Please watch the demos to see the changes I made:

Before:

After:
iOS:

Android:

Reviewers: atul, rohan

Reviewed By: atul, rohan

Subscribers: ashoat, tomek, atul, abosh

Differential Revision: https://phab.comm.dev/D5524

Details

Provenance
ginsuAuthored on Oct 31 2022, 8:28 AM
Reviewer
atul
Differential Revision
D5524: [native] introduced more action to tooltip actions
Parents
rCOMM1eca05c1ada4: [native] Moved fixed tooltip to the bottom of chat window
Branches
Unknown
Tags
Unknown