Skip to content

Commit

Permalink
[native] add farcaster icon on native
Browse files Browse the repository at this point in the history
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
  • Loading branch information
vdhanan committed Sep 26, 2024
1 parent b7a9f62 commit 18fde5a
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion native/avatars/avatar-hooks.js
Original file line number Diff line number Diff line change
Expand Up @@ -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}
/>
);
Expand Down

0 comments on commit 18fde5a

Please sign in to comment.