Page MenuHomePhabricator
Differential D9289

[native] make user profile accessible from message author avatar
ClosedPublic
Actions

Authored by ginsu on Sep 25 2023, 7:21 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 15, 7:59 AM
Unknown Object (File)
Fri, Nov 8, 11:27 AM
Unknown Object (File)
Fri, Nov 1, 8:43 PM
Unknown Object (File)
Fri, Nov 1, 8:43 PM
Unknown Object (File)
Fri, Nov 1, 8:43 PM
Unknown Object (File)
Fri, Nov 1, 8:36 PM
Unknown Object (File)
Tue, Oct 29, 2:18 PM
Unknown Object (File)
Oct 2 2024, 10:28 AM
View All 99 Files
Subscribers
ashoat
tomek

Details

Reviewers
atul
inka
Commits
rCOMM6e296d4e4291: [native] make user profile accessible from message author avatar
Summary

This diff makes the user avatar in the message thread a TouchableOpacity that will navigate to the user profile whenever onPress is triggered

Depends on D9286

Test Plan

Please see the demo video below

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu created this revision.Sep 25 2023, 7:21 AM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptSep 25 2023, 7:21 AM
ginsu edited the summary of this revision. (Show Details)Sep 25 2023, 7:23 AM
ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: atul, inka.
ginsu added a parent revision: D9286: [native] make user profiles accessible from RelationshipListItem.
ginsu edited the test plan for this revision. (Show Details)
ginsu attached a referenced file: F773465: Screen Recording 2023-09-25 at 10.23.52 AM.mov. (Show Details)
Harbormaster completed remote builds in B22832: Diff 31403.Sep 25 2023, 7:38 AM
ginsu requested review of this revision.Sep 25 2023, 7:38 AM
atul accepted this revision.Sep 26 2023, 2:12 PM

Looks good!

This revision is now accepted and ready to land.Sep 26 2023, 2:12 PM
ginsu added a child revision: D9385: [native] utilize memoization in the MesssageHeader component.Oct 6 2023, 3:58 AM
ginsu updated this revision to Diff 31889.Oct 11 2023, 1:31 AM

rebase before landing

Harbormaster completed remote builds in B23158: Diff 31889.Oct 11 2023, 1:49 AM
Closed by commit rCOMM6e296d4e4291: [native] make user profile accessible from message author avatar (authored by ginsu). · Explain WhyOct 11 2023, 2:39 AM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM6e296d4e4291: [native] make user profile accessible from message author avatar.

Revision Contents
Changeset List

PathSize
native/
chat/
28 lines

Diff 31403

View Options

native/chat/composed-message.react.js

Loading...