Page MenuHomePhabricator
Differential D14112

[native] Move render function from Tooltip to functional component
ClosedPublic
Actions

Authored by angelika on Wed, Dec 11, 7:59 AM.
Tags
None
Referenced Files
F3488506: D14112.id46409.diff
Wed, Dec 18, 10:15 AM
Unknown Object (File)
Tue, Dec 17, 3:33 PM
Unknown Object (File)
Tue, Dec 17, 2:55 PM
Unknown Object (File)
Fri, Dec 13, 11:17 PM
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMM0f683239b037: [native] Move render function from Tooltip to functional component
Summary

Move render() function to the functional compoent as a part of Tooltip migration to functional component. This finishes the migration of the Tooltip component to functional component.

Depends on D14111

Test Plan

Open up some tooltips and make sure they look correct.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

angelika created this revision.Wed, Dec 11, 7:59 AM
angelika held this revision as a draft.
angelika added a child revision: D14113: [native] Enable Flow types in Reanimated worklets.Wed, Dec 11, 8:14 AM
Harbormaster completed remote builds in B33012: Diff 46330.Wed, Dec 11, 9:29 AM
angelika published this revision for review.Wed, Dec 11, 4:55 PM
ashoat accepted this revision.Wed, Dec 11, 5:57 PM
ashoat added inline comments.
native/tooltip/tooltip.react.js
150 ↗(On Diff #46330)

Nit: can we rename this to just Tooltip now?

This revision is now accepted and ready to land.Wed, Dec 11, 5:57 PM
ashoat added inline comments.Wed, Dec 11, 8:05 PM
native/tooltip/tooltip.react.js
128 ↗(On Diff #46330)

This can be renamed now

angelika updated this revision to Diff 46409.Fri, Dec 13, 5:33 AM

Rebase and address feedback

angelika marked 2 inline comments as done.Fri, Dec 13, 6:17 AM
Harbormaster completed remote builds in B33087: Diff 46409.Fri, Dec 13, 7:07 AM
angelika updated this revision to Diff 46472.Mon, Dec 16, 6:21 AM

Rebase

Harbormaster completed remote builds in B33131: Diff 46472.Mon, Dec 16, 8:01 AM
Closed by commit rCOMM0f683239b037: [native] Move render function from Tooltip to functional component (authored by angelika). · Explain WhyMon, Dec 16, 1:34 PM
This revision was automatically updated to reflect the committed changes.
angelika added a commit: rCOMM0f683239b037: [native] Move render function from Tooltip to functional component.

Revision Contents
Changeset List

Diff 46508

View Options

native/chat/multimedia-message-tooltip-modal.react.js

Loading...
View Options

native/chat/robotext-message-tooltip-modal.react.js

Loading...
View Options

native/chat/settings/thread-settings-member-tooltip-modal.react.js

Loading...
View Options

native/chat/text-message-tooltip-modal.react.js

Loading...
View Options

native/profile/user-relationship-tooltip-modal.react.js

Loading...
View Options

native/tooltip/tooltip.react.js

Loading...