Page MenuHomePhabricator
Differential D14119

[native] Migrate messageContainerStyle in useAnimatedMessageTooltipButton to Reanimate V2 API
ClosedPublic
Actions

Authored by angelika on Dec 11 2024, 8:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 3, 6:21 AM
Unknown Object (File)
Tue, Feb 25, 3:16 AM
Unknown Object (File)
Thu, Feb 20, 2:25 AM
Unknown Object (File)
Tue, Feb 18, 10:07 AM
Unknown Object (File)
Wed, Feb 12, 7:02 PM
Unknown Object (File)
Wed, Feb 12, 7:01 PM
Unknown Object (File)
Wed, Feb 12, 7:01 PM
Unknown Object (File)
Mon, Feb 10, 5:42 PM
View All 59 Files
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMM76af728667fe: [native] Migrate messageContainerStyle in useAnimatedMessageTooltipButton to…
Summary

This is a straightforward change: use progressV2 variable, use useAnimatedStyle instead of useMemo for styles and put the interpolation in useAnimatedStyle worklet.

Depends on D14118

Test Plan

Open up text message tooltip and verify it's animated correctly. Also navigate to the thread from the tooltip and verify the transition looks right.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

angelika created this revision.Dec 11 2024, 8:33 AM
angelika held this revision as a draft.
angelika added a child revision: D14120: [native] Migrate threadColorOverride in useAnimatedMessageTooltipButton to Reanimate V2 API.Dec 11 2024, 8:46 AM
Harbormaster completed remote builds in B33019: Diff 46337.Dec 11 2024, 10:40 AM
angelika published this revision for review.Dec 11 2024, 5:27 PM
ashoat accepted this revision.Dec 11 2024, 7:51 PM
This revision is now accepted and ready to land.Dec 11 2024, 7:51 PM
angelika updated this revision to Diff 46413.Dec 13 2024, 5:35 AM

Rebase and address feedback

Harbormaster completed remote builds in B33091: Diff 46413.Dec 13 2024, 7:40 AM
angelika updated this revision to Diff 46476.Dec 16 2024, 6:22 AM

Rebase

Harbormaster completed remote builds in B33135: Diff 46476.Dec 16 2024, 8:32 AM
Closed by commit rCOMM76af728667fe: [native] Migrate messageContainerStyle in useAnimatedMessageTooltipButton to… (authored by angelika). · Explain WhyDec 16 2024, 1:34 PM
This revision was automatically updated to reflect the committed changes.
angelika added a commit: rCOMM76af728667fe: [native] Migrate messageContainerStyle in useAnimatedMessageTooltipButton to….

Revision Contents
Changeset List

PathSize
native/
chat/
27 lines

Diff 46413

View Options

native/chat/utils.js

Loading...