Skip to content

Commit

Permalink
BottomSheet: Setting DashIcon color directly when theme is default (l…
Browse files Browse the repository at this point in the history
…ight) (#17193)
  • Loading branch information
etoledom committed Aug 26, 2019
1 parent fe4c02f commit bf84833
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
4 changes: 3 additions & 1 deletion packages/components/src/mobile/bottom-sheet/cell.native.js
Original file line number Diff line number Diff line change
Expand Up @@ -151,6 +151,8 @@ class BottomSheetCell extends Component {
);
};

const iconStyle = getStyle( styles.icon, styles.iconDark, theme );

return (
<TouchableOpacity
accessible={ ! this.state.isEditingValue }
Expand All @@ -168,7 +170,7 @@ class BottomSheetCell extends Component {
<View style={ styles.cellRowContainer }>
{ icon && (
<View style={ styles.cellRowContainer }>
<Dashicon icon={ icon } size={ 24 } color={ theme === 'dark' && '#C3C4C7' } />
<Dashicon icon={ icon } size={ 24 } color={ iconStyle.color } />
<View style={ platformStyles.labelIconSeparator } />
</View>
) }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -134,3 +134,11 @@
.cellValueRTL {
text-align: left;
}

.icon {
color: #7b9ab1;
}

.iconDark {
color: #c3c4c7;
}

0 comments on commit bf84833

Please sign in to comment.