Page MenuHomePhabricator
Differential D13476

[native] add farcaster icon on native
ClosedPublic
Actions

Authored by varun on Wed, Sep 25, 5:09 PM.
Tags
None
Referenced Files
F2838221: D13476.diff
Sat, Sep 28, 9:21 PM
Unknown Object (File)
Fri, Sep 27, 6:41 PM
Unknown Object (File)
Fri, Sep 27, 8:20 AM
Unknown Object (File)
Thu, Sep 26, 1:18 AM
Unknown Object (File)
Thu, Sep 26, 1:18 AM
Unknown Object (File)
Thu, Sep 26, 1:15 AM
Unknown Object (File)
Thu, Sep 26, 1:11 AM
F2810498: Screenshot_1727307971.png
Wed, Sep 25, 5:14 PM
View All 9 Files
Subscribers
ashoat
tomek

Details

Reviewers
will
Commits
rCOMM18fde5a6777d: [native] add farcaster icon on native
Summary

missed this in my previous diff stack. we need to add the farcaster icon to icons to make it available in the action sheet.

i also resized the ethereum-outline icon because it looked too small

Depends on D13475

Test Plan

see attached photos. farcaster icon now appears in action sheet and the ethereum icon looks right

Screenshot_1727308308.png (3×1 px, 163 KB)

Screenshot_1727307971.png (3×1 px, 162 KB)

(simply replaced the icon name to test the ethereum icon size, not actually using that icon for the farcaster avatar option in action sheet)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun created this revision.Wed, Sep 25, 5:09 PM
varun held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptWed, Sep 25, 5:09 PM
varun edited the test plan for this revision. (Show Details)Wed, Sep 25, 5:14 PM
varun attached a referenced file: F2810498: Screenshot_1727307971.png. (Show Details)
varun attached a referenced file: F2810499: Screenshot_1727308308.png. (Show Details)
varun published this revision for review.Wed, Sep 25, 5:22 PM

will wait for CI to pass before landing

will accepted this revision.Wed, Sep 25, 5:22 PM
This revision is now accepted and ready to land.Wed, Sep 25, 5:22 PM
Harbormaster completed remote builds in B31862: Diff 44577.Wed, Sep 25, 5:27 PM
Closed by commit rCOMM18fde5a6777d: [native] add farcaster icon on native (authored by varun). · Explain WhyWed, Sep 25, 5:29 PM
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMM18fde5a6777d: [native] add farcaster icon on native.
ashoat added a comment.Wed, Sep 25, 5:54 PM

Why does the home pill appear over the options? Do we need to wrap something in a SafeAreaView?

varun added a comment.EditedWed, Sep 25, 8:52 PM
In D13476#378823, @ashoat wrote:

Why does the home pill appear over the options? Do we need to wrap something in a SafeAreaView?

thanks for pointing that out. D13477

Revision Contents
Changeset List

PathSize
native/
avatars/
11 lines

Diff 44579

View Options

native/avatars/avatar-hooks.js

Loading...