Skip to content

Commit

Permalink
Merge pull request #19597 from Skalakid/17015-migrate-SidebarLinks
Browse files Browse the repository at this point in the history
17015 - migrate SidebarLinks to PressableWithoutFeedback
  • Loading branch information
arosiclair authored Jun 5, 2023
2 parents 132358f + 32985d5 commit f36d9f2
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions src/pages/home/sidebar/SidebarLinks.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
/* eslint-disable rulesdir/onyx-props-must-have-default */
import lodashGet from 'lodash/get';
import React from 'react';
import {View, TouchableOpacity} from 'react-native';
import {View} from 'react-native';
import _ from 'underscore';
import PropTypes from 'prop-types';
import {withOnyx} from 'react-native-onyx';
Expand Down Expand Up @@ -33,6 +33,7 @@ import defaultTheme from '../../../styles/themes/default';
import OptionsListSkeletonView from '../../../components/OptionsListSkeletonView';
import variables from '../../../styles/variables';
import LogoComponent from '../../../../assets/images/expensify-wordmark.svg';
import PressableWithoutFeedback from '../../../components/Pressable/PressableWithoutFeedback';
import * as Session from '../../../libs/actions/Session';
import Button from '../../../components/Button';
import * as UserUtils from '../../../libs/UserUtils';
Expand Down Expand Up @@ -166,16 +167,16 @@ class SidebarLinks extends React.Component {
shouldShowEnvironmentBadge
/>
<Tooltip text={this.props.translate('common.search')}>
<TouchableOpacity
<PressableWithoutFeedback
accessibilityLabel={this.props.translate('sidebarScreen.buttonSearch')}
accessibilityRole="button"
style={[styles.flexRow, styles.ph5]}
onPress={Session.checkIfActionIsAllowed(this.showSearchPage)}
>
<Icon src={Expensicons.MagnifyingGlass} />
</TouchableOpacity>
</PressableWithoutFeedback>
</Tooltip>
<TouchableOpacity
<PressableWithoutFeedback
accessibilityLabel={this.props.translate('sidebarScreen.buttonMySettings')}
accessibilityRole="button"
onPress={Session.checkIfActionIsAllowed(this.showSettingsPage)}
Expand All @@ -197,7 +198,7 @@ class SidebarLinks extends React.Component {
/>
</OfflineWithFeedback>
)}
</TouchableOpacity>
</PressableWithoutFeedback>
</View>
<Freeze
freeze={shouldFreeze}
Expand Down

0 comments on commit f36d9f2

Please sign in to comment.