Page MenuHomePhabricator
Differential D14131

[native] Move view ref in MultimediaMessageMultimedia to functional component
ClosedPublic
Actions

Authored by angelika on Dec 11 2024, 12:07 PM.
Tags
None
Referenced Files
F3760408: D14131.diff
Fri, Jan 10, 1:44 PM
Unknown Object (File)
Wed, Jan 8, 10:37 AM
Unknown Object (File)
Tue, Jan 7, 8:47 AM
Unknown Object (File)
Mon, Jan 6, 1:26 PM
Unknown Object (File)
Sat, Jan 4, 6:00 PM
Unknown Object (File)
Sat, Dec 28, 9:18 PM
Unknown Object (File)
Sat, Dec 28, 3:02 PM
Unknown Object (File)
Sat, Dec 28, 12:36 AM
View All 40 Files
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMMfcbbc62e5707: [native] Move view ref in MultimediaMessageMultimedia to functional component
Summary

Move view ref to functional component as a part of MultimediaMessageMultimedia migration to a functional component.

Depends on D14130

Test Plan

Open up a multimedia message and see if it behaves correctly.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

angelika created this revision.Dec 11 2024, 12:07 PM
angelika held this revision as a draft.
angelika edited the summary of this revision. (Show Details)Dec 11 2024, 12:08 PM
angelika added a parent revision: D14130: [native] Migrate ActionResultModal component to Reanimated v2 API.
angelika added a child revision: D14132: [native] Move onPress and onLayout methods in MultimediaMessageMultimedia to functional component.Dec 11 2024, 12:14 PM
Harbormaster completed remote builds in B33032: Diff 46350.Dec 11 2024, 12:48 PM
angelika published this revision for review.Dec 11 2024, 5:29 PM
ashoat accepted this revision.Wed, Dec 11, 8:02 PM
This revision is now accepted and ready to land.Wed, Dec 11, 8:02 PM
angelika updated this revision to Diff 46425.Fri, Dec 13, 5:41 AM

Rebase and address feedback

Harbormaster completed remote builds in B33103: Diff 46425.Fri, Dec 13, 9:21 AM
angelika updated this revision to Diff 46488.Mon, Dec 16, 6:24 AM

Rebase

Harbormaster completed remote builds in B33147: Diff 46488.Mon, Dec 16, 10:18 AM
Closed by commit rCOMMfcbbc62e5707: [native] Move view ref in MultimediaMessageMultimedia to functional component (authored by angelika). · Explain WhyMon, Dec 16, 1:35 PM
This revision was automatically updated to reflect the committed changes.
angelika added a commit: rCOMMfcbbc62e5707: [native] Move view ref in MultimediaMessageMultimedia to functional component.

Revision Contents
Changeset List

PathSize
native/
chat/
18 lines

Diff 46524

View Options

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

Loading...