Skip to content

Commit

Permalink
fix color for focused item in global navigation
Browse files Browse the repository at this point in the history
  • Loading branch information
adamgrzybowski committed Oct 5, 2023
1 parent 8bb5406 commit 4c056fa
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import * as StyleUtils from '../../../../styles/StyleUtils';
import Icon from '../../../../components/Icon';
import CONST from '../../../../CONST';
import variables from '../../../../styles/variables';
import defaultTheme from '../../../../styles/themes/default';
import PressableWithFeedback from '../../../../components/Pressable/PressableWithFeedback';
import refPropTypes from '../../../../components/refPropTypes';

Expand Down Expand Up @@ -53,7 +52,7 @@ function GlobalNavigationMenuItem({icon, title, isFocused, onPress, forwardedRef
additionalStyles={[styles.popoverMenuIcon]}
pressed={pressed}
src={icon}
fill={isFocused ? defaultTheme.iconMenu : defaultTheme.icon}
fill={isFocused ? StyleUtils.getIconFillColor(CONST.BUTTON_STATES.DEFAULT, true) : StyleUtils.getIconFillColor()}
/>
<View style={[styles.mt1, styles.alignItemsCenter]}>
<Text style={[StyleUtils.getFontSizeStyle(variables.fontSizeExtraSmall), styles.globalNavigationMenuItem(isFocused)]}>{title}</Text>
Expand Down
2 changes: 1 addition & 1 deletion src/styles/styles.js
Original file line number Diff line number Diff line change
Expand Up @@ -3734,7 +3734,7 @@ const styles = (theme) => ({
backgroundColor: isFocused ? theme.iconMenu : theme.transparent,
}),

globalNavigationMenuItem: (isFocused) => (isFocused ? {color: theme.textLight, fontWeight: fontWeightBold, fontFamily: fontFamily.EXP_NEUE_BOLD} : {color: theme.icon}),
globalNavigationMenuItem: (isFocused) => (isFocused ? {color: theme.text, fontWeight: fontWeightBold, fontFamily: fontFamily.EXP_NEUE_BOLD} : {color: theme.icon}),

globalNavigationItemContainer: {
width: variables.globalNavigationWidth,
Expand Down

0 comments on commit 4c056fa

Please sign in to comment.