Skip to content

Commit ee3fb21

Browse files
luacmartinsOSBotify
authored andcommitted
Merge pull request #51032 from software-mansion-labs/kicu/50982-search-fixes
[CP Staging] Fix Search back button on mWeb and report opening in Search (cherry picked from commit 20b85d5) (CP triggered by luacmartins)
1 parent 9be24c1 commit ee3fb21

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

src/components/Search/SearchRouter/SearchRouterList.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ function SearchRouterList(
167167
if ('reportID' in item && item?.reportID) {
168168
Navigation.navigate(ROUTES.REPORT_WITH_ID.getRoute(item?.reportID));
169169
} else if ('login' in item) {
170-
Report.navigateToAndOpenReport(item?.login ? [item.login] : []);
170+
Report.navigateToAndOpenReport(item.login ? [item.login] : [], false);
171171
}
172172
},
173173
[closeAndClearRouter, onSearchSubmit, currentQuery, updateUserSearchQuery],

src/components/Search/SearchRouter/SearchRouterModal.tsx

+8-2
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import React from 'react';
22
import FocusTrapForModal from '@components/FocusTrap/FocusTrapForModal';
33
import Modal from '@components/Modal';
44
import useResponsiveLayout from '@hooks/useResponsiveLayout';
5+
import * as Browser from '@libs/Browser';
56
import CONST from '@src/CONST';
67
import SearchRouter from './SearchRouter';
78
import {useSearchRouterContext} from './SearchRouterContext';
@@ -15,12 +16,17 @@ function SearchRouterModal() {
1516
return (
1617
<Modal
1718
type={modalType}
18-
fullscreen
1919
isVisible={isSearchRouterDisplayed}
2020
popoverAnchorPosition={{right: 6, top: 6}}
21+
fullscreen
22+
shouldHandleNavigationBack={Browser.isMobileChrome()}
2123
onClose={closeSearchRouter}
2224
>
23-
<FocusTrapForModal active={isSearchRouterDisplayed}>{isSearchRouterDisplayed && <SearchRouter onRouterClose={closeSearchRouter} />}</FocusTrapForModal>
25+
{isSearchRouterDisplayed && (
26+
<FocusTrapForModal active={isSearchRouterDisplayed}>
27+
<SearchRouter onRouterClose={closeSearchRouter} />
28+
</FocusTrapForModal>
29+
)}
2430
</Modal>
2531
);
2632
}

0 commit comments

Comments
 (0)