Page MenuHomePhabricator
Differential D14132

[native] Move onPress and onLayout methods in MultimediaMessageMultimedia to functional component
ClosedPublic
Actions

Authored by angelika on Wed, Dec 11, 12:14 PM.
Tags
None
Referenced Files
F3508499: D14132.id46351.diff
Fri, Dec 20, 10:31 PM
F3506115: D14132.id46426.diff
Fri, Dec 20, 3:38 PM
F3505920: D14132.id46490.diff
Fri, Dec 20, 3:01 PM
F3505884: D14132.id46351.diff
Fri, Dec 20, 2:56 PM
F3505568: D14132.id46489.diff
Fri, Dec 20, 1:53 PM
F3504715: D14132.id46426.diff
Fri, Dec 20, 10:15 AM
Unknown Object (File)
Fri, Dec 20, 5:55 AM
Unknown Object (File)
Thu, Dec 19, 6:55 AM
View All 15 Files
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMMe8c3300a219b: [native] Move onPress and onLayout methods in MultimediaMessageMultimedia to…
Summary

Move onPress and onLayout methods to functional component as a part of MultimediaMessageMultimedia migration to a functional component. Empty onLayout looks weird but in the history I can see it was introduced to fix some Android measure bug and I believe it so I'm leaving it as it was.

Depends on D14131

Test Plan

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

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

angelika created this revision.Wed, Dec 11, 12:14 PM
angelika held this revision as a draft.
angelika added a child revision: D14133: [native] Migrate opacity in MultimediaMessageMultimedia to Reanimated v2 API in functional component.Wed, Dec 11, 12:26 PM
Harbormaster completed remote builds in B33033: Diff 46351.Wed, Dec 11, 12:57 PM
angelika published this revision for review.Wed, Dec 11, 5:30 PM
ashoat accepted this revision.Wed, Dec 11, 8:03 PM
This revision is now accepted and ready to land.Wed, Dec 11, 8:03 PM
angelika updated this revision to Diff 46426.Fri, Dec 13, 5:42 AM

Rebase and address feedback

Harbormaster completed remote builds in B33104: Diff 46426.Fri, Dec 13, 9:29 AM
angelika updated this revision to Diff 46489.Mon, Dec 16, 6:25 AM

Rebase

Harbormaster completed remote builds in B33148: Diff 46489.Mon, Dec 16, 10:26 AM
Closed by commit rCOMMe8c3300a219b: [native] Move onPress and onLayout methods in MultimediaMessageMultimedia to… (authored by angelika). · Explain WhyMon, Dec 16, 1:35 PM
This revision was automatically updated to reflect the committed changes.
angelika added a commit: rCOMMe8c3300a219b: [native] Move onPress and onLayout methods in MultimediaMessageMultimedia to….

Revision Contents
Changeset List

PathSize
native/
chat/
105 lines

Diff 46351

View Options

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

Loading...