Paths

Table of Contentst

Differential D13476

[native] add farcaster icon on native
ClosedPublic
Actions

Authored by varun on Sep 25 2024, 5:09 PM.

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

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Sep 25 2024, 5:09 PM

varun held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptSep 25 2024, 5:09 PM

varun edited the test plan for this revision. (Show Details)Sep 25 2024, 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)

will wait for CI to pass before landing

will accepted this revision.Sep 25 2024, 5:22 PM

This revision is now accepted and ready to land.Sep 25 2024, 5:22 PM

Harbormaster completed remote builds in B31862: Diff 44577.Sep 25 2024, 5:27 PM

Closed by commit rCOMM18fde5a6777d: [native] add farcaster icon on native (authored by varun). · Explain WhySep 25 2024, 5:29 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM18fde5a6777d: [native] add farcaster icon on native.

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

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

Path

Size

native/

avatars/

avatar-hooks.js

11 lines

Diff 44577

View Options

native/avatars/avatar-hooks.js

[native] add farcaster icon on nativeClosedPublicActions