diff options
author | Eric Bailey <git@esb.lol> | 2024-05-10 10:42:45 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-05-10 10:42:45 -0500 |
commit | 8f56f79c6c94a7adf1de304097067f5aed0a111a (patch) | |
tree | e5ffc40892986329385ba60d0d2836ae2e0069fc /src/view/shell/bottom-bar/BottomBar.tsx | |
parent | d7f3a8d01fffafac0841bcb732bff0d4a7e53f01 (diff) | |
download | voidsky-8f56f79c6c94a7adf1de304097067f5aed0a111a.tar.zst |
[🐴] Change up icons (#3938)
* Swap for chat icon * Replace icons in left nav * Replace icons in bottom bars * Ditch feeds, drop size * Fine tune * Swap bell icon, improve alignment and size
Diffstat (limited to 'src/view/shell/bottom-bar/BottomBar.tsx')
-rw-r--r-- | src/view/shell/bottom-bar/BottomBar.tsx | 148 |
1 files changed, 57 insertions, 91 deletions
diff --git a/src/view/shell/bottom-bar/BottomBar.tsx b/src/view/shell/bottom-bar/BottomBar.tsx index 0db8b242a..bcaaf3417 100644 --- a/src/view/shell/bottom-bar/BottomBar.tsx +++ b/src/view/shell/bottom-bar/BottomBar.tsx @@ -13,15 +13,6 @@ import {useDedupe} from '#/lib/hooks/useDedupe' import {useMinimalShellMode} from '#/lib/hooks/useMinimalShellMode' import {useNavigationTabState} from '#/lib/hooks/useNavigationTabState' import {usePalette} from '#/lib/hooks/usePalette' -import { - BellIcon, - BellIconSolid, - HashtagIcon, - HomeIcon, - HomeIconSolid, - MagnifyingGlassIcon2, - MagnifyingGlassIcon2Solid, -} from '#/lib/icons' import {clamp} from '#/lib/numbers' import {getTabState, TabState} from '#/lib/routes/helpers' import {useGate} from '#/lib/statsig/statsig' @@ -41,8 +32,20 @@ import {Logo} from '#/view/icons/Logo' import {Logotype} from '#/view/icons/Logotype' import {useDialogControl} from '#/components/Dialog' import {SwitchAccountDialog} from '#/components/dialogs/SwitchAccount' -import {Envelope_Stroke2_Corner0_Rounded as Envelope} from '#/components/icons/Envelope' -import {Envelope_Filled_Stroke2_Corner0_Rounded as EnvelopeFilled} from '#/components/icons/Envelope' +import { + Bell_Filled_Corner0_Rounded as BellFilled, + Bell_Stroke2_Corner0_Rounded as Bell, +} from '#/components/icons/Bell' +import { + HomeOpen_Filled_Corner0_Rounded as HomeFilled, + HomeOpen_Stoke2_Corner0_Rounded as Home, +} from '#/components/icons/HomeOpen' +import {MagnifyingGlass_Filled_Stroke2_Corner0_Rounded as MagnifyingGlassFilled} from '#/components/icons/MagnifyingGlass' +import {MagnifyingGlass2_Stroke2_Corner0_Rounded as MagnifyingGlass} from '#/components/icons/MagnifyingGlass2' +import { + Message_Stroke2_Corner0_Rounded as Message, + Message_Stroke2_Corner0_Rounded_Filled as MessageFilled, +} from '#/components/icons/Message' import {styles} from './BottomBarStyles' type TabOptions = @@ -60,14 +63,8 @@ export function BottomBar({navigation}: BottomTabBarProps) { const safeAreaInsets = useSafeAreaInsets() const {track} = useAnalytics() const {footerHeight} = useShellLayout() - const { - isAtHome, - isAtSearch, - isAtFeeds, - isAtNotifications, - isAtMyProfile, - isAtMessages, - } = useNavigationTabState() + const {isAtHome, isAtSearch, isAtNotifications, isAtMyProfile, isAtMessages} = + useNavigationTabState() const numUnreadNotifications = useUnreadNotifications() const numUnreadMessages = useUnreadMessageCount() const {footerMinimalShellTransform} = useMinimalShellMode() @@ -78,6 +75,7 @@ export function BottomBar({navigation}: BottomTabBarProps) { const accountSwitchControl = useDialogControl() const playHaptic = useHaptics() const gate = useGate() + const iconWidth = 28 const showSignIn = React.useCallback(() => { closeAllActiveElements() @@ -110,10 +108,6 @@ export function BottomBar({navigation}: BottomTabBarProps) { () => onPressTab('Search'), [onPressTab], ) - const onPressFeeds = React.useCallback( - () => onPressTab('Feeds'), - [onPressTab], - ) const onPressNotifications = React.useCallback( () => onPressTab('Notifications'), [onPressTab], @@ -152,15 +146,13 @@ export function BottomBar({navigation}: BottomTabBarProps) { testID="bottomBarHomeBtn" icon={ isAtHome ? ( - <HomeIconSolid - strokeWidth={4} - size={24} + <HomeFilled + width={iconWidth + 1} style={[styles.ctrlIcon, pal.text, styles.homeIcon]} /> ) : ( - <HomeIcon - strokeWidth={4} - size={24} + <Home + width={iconWidth + 1} style={[styles.ctrlIcon, pal.text, styles.homeIcon]} /> ) @@ -174,16 +166,14 @@ export function BottomBar({navigation}: BottomTabBarProps) { testID="bottomBarSearchBtn" icon={ isAtSearch ? ( - <MagnifyingGlassIcon2Solid - size={25} + <MagnifyingGlassFilled + width={iconWidth + 2} style={[styles.ctrlIcon, pal.text, styles.searchIcon]} - strokeWidth={1.8} /> ) : ( - <MagnifyingGlassIcon2 - size={25} + <MagnifyingGlass + width={iconWidth + 2} style={[styles.ctrlIcon, pal.text, styles.searchIcon]} - strokeWidth={1.8} /> ) } @@ -192,68 +182,18 @@ export function BottomBar({navigation}: BottomTabBarProps) { accessibilityLabel={_(msg`Search`)} accessibilityHint="" /> - <Btn - testID="bottomBarFeedsBtn" - icon={ - isAtFeeds ? ( - <HashtagIcon - size={24} - style={[styles.ctrlIcon, pal.text, styles.feedsIcon]} - strokeWidth={4} - /> - ) : ( - <HashtagIcon - size={24} - style={[styles.ctrlIcon, pal.text, styles.feedsIcon]} - strokeWidth={2.25} - /> - ) - } - onPress={onPressFeeds} - accessibilityRole="tab" - accessibilityLabel={_(msg`Feeds`)} - accessibilityHint="" - /> - <Btn - testID="bottomBarNotificationsBtn" - icon={ - isAtNotifications ? ( - <BellIconSolid - size={24} - strokeWidth={1.9} - style={[styles.ctrlIcon, pal.text, styles.bellIcon]} - /> - ) : ( - <BellIcon - size={24} - strokeWidth={1.9} - style={[styles.ctrlIcon, pal.text, styles.bellIcon]} - /> - ) - } - onPress={onPressNotifications} - notificationCount={numUnreadNotifications} - accessible={true} - accessibilityRole="tab" - accessibilityLabel={_(msg`Notifications`)} - accessibilityHint={ - numUnreadNotifications === '' - ? '' - : `${numUnreadNotifications} unread` - } - /> {gate('dms') && ( <Btn testID="bottomBarMessagesBtn" icon={ isAtMessages ? ( - <EnvelopeFilled - size="lg" + <MessageFilled + width={iconWidth - 1} style={[styles.ctrlIcon, pal.text, styles.feedsIcon]} /> ) : ( - <Envelope - size="lg" + <Message + width={iconWidth - 1} style={[styles.ctrlIcon, pal.text, styles.feedsIcon]} /> ) @@ -271,6 +211,32 @@ export function BottomBar({navigation}: BottomTabBarProps) { /> )} <Btn + testID="bottomBarNotificationsBtn" + icon={ + isAtNotifications ? ( + <BellFilled + width={iconWidth} + style={[styles.ctrlIcon, pal.text, styles.bellIcon]} + /> + ) : ( + <Bell + width={iconWidth} + style={[styles.ctrlIcon, pal.text, styles.bellIcon]} + /> + ) + } + onPress={onPressNotifications} + notificationCount={numUnreadNotifications} + accessible={true} + accessibilityRole="tab" + accessibilityLabel={_(msg`Notifications`)} + accessibilityHint={ + numUnreadNotifications === '' + ? '' + : `${numUnreadNotifications} unread` + } + /> + <Btn testID="bottomBarProfileBtn" icon={ <View style={styles.ctrlIconSizingWrapper}> @@ -285,7 +251,7 @@ export function BottomBar({navigation}: BottomTabBarProps) { ]}> <UserAvatar avatar={profile?.avatar} - size={27} + size={iconWidth - 3} // See https://github.com/bluesky-social/social-app/pull/1801: usePlainRNImage={true} type={profile?.associated?.labeler ? 'labeler' : 'user'} @@ -296,7 +262,7 @@ export function BottomBar({navigation}: BottomTabBarProps) { style={[styles.ctrlIcon, pal.text, styles.profileIcon]}> <UserAvatar avatar={profile?.avatar} - size={28} + size={iconWidth - 3} // See https://github.com/bluesky-social/social-app/pull/1801: usePlainRNImage={true} type={profile?.associated?.labeler ? 'labeler' : 'user'} |