From 18fde5a6777dcabef55faaf4b63a0d9e3a0c5aea Mon Sep 17 00:00:00 2001
From: Varun Dhananjaya <varun.dhananjaya@gmail.com>
Date: Wed, 25 Sep 2024 19:59:08 -0400
Subject: [PATCH] [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

{F2810499}

{F2810498}

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

Reviewers: will

Reviewed By: will

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D13476
---
 native/avatars/avatar-hooks.js | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/native/avatars/avatar-hooks.js b/native/avatars/avatar-hooks.js
index 9e9a053ed4..601c8c924b 100644
--- a/native/avatars/avatar-hooks.js
+++ b/native/avatars/avatar-hooks.js
@@ -541,7 +541,16 @@ function useShowAvatarActionSheet(
           <CommIcon
             name="ethereum-outline"
             key={option.id}
-            size={18}
+            size={22}
+            style={styles.bottomSheetIcon}
+          />
+        );
+      } else if (option.id === 'farcaster') {
+        return (
+          <CommIcon
+            name="farcaster-outline"
+            key={option.id}
+            size={22}
             style={styles.bottomSheetIcon}
           />
         );