Skip to content

Commit c512e63

Browse files
committed
Hide the user search option for mobile devices
1 parent c011aad commit c512e63

File tree

3 files changed

+19
-16
lines changed

3 files changed

+19
-16
lines changed

apps/app/src/features/search/client/components/SearchPage/SearchControl.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -212,6 +212,7 @@ const SearchControl = React.memo((props: Props): JSX.Element => {
212212
<SearchOptionModalLazyLoaded
213213
isOpen={isFileterOptionModalShown || false}
214214
onClose={() => setIsFileterOptionModalShown(false)}
215+
isHidingUserPages={isHidingUserPages}
215216
includeUserPages={includeUserPages}
216217
includeTrashPages={includeTrashPages}
217218
onIncludeUserPagesSwitched={setIncludeUserPages}

apps/app/src/features/search/client/components/SearchPage/SearchOptionModal/SearchOptionModal.tsx

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ type Props = {
66
isOpen: boolean;
77
includeUserPages: boolean;
88
includeTrashPages: boolean;
9+
isHidingUserPages: boolean;
910
onClose?: () => void;
1011
onIncludeUserPagesSwitched?: (isChecked: boolean) => void;
1112
onIncludeTrashPagesSwitched?: (isChecked: boolean) => void;
@@ -18,6 +19,7 @@ export const SearchOptionModal: FC<Props> = (props: Props) => {
1819
isOpen,
1920
includeUserPages,
2021
includeTrashPages,
22+
isHidingUserPages,
2123
onClose,
2224
onIncludeUserPagesSwitched,
2325
onIncludeTrashPagesSwitched,
@@ -31,9 +33,9 @@ export const SearchOptionModal: FC<Props> = (props: Props) => {
3133
}, [onClose]);
3234

3335
const includeUserPagesChangeHandler = useCallback(
34-
(isChecked: boolean) => {
36+
(e: React.ChangeEvent<HTMLInputElement>) => {
3537
if (onIncludeUserPagesSwitched != null) {
36-
onIncludeUserPagesSwitched(isChecked);
38+
onIncludeUserPagesSwitched(e.target.checked);
3739
}
3840
},
3941
[onIncludeUserPagesSwitched],
@@ -55,20 +57,19 @@ export const SearchOptionModal: FC<Props> = (props: Props) => {
5557
</ModalHeader>
5658
<ModalBody>
5759
<div className="d-flex p-2">
58-
<div className="me-3">
59-
<label className="form-label px-3 py-2 mb-0 d-flex align-items-center">
60-
<input
61-
className="me-2"
62-
type="checkbox"
63-
onChange={useCallback(
64-
(e) => includeUserPagesChangeHandler(e.target.checked),
65-
[includeUserPagesChangeHandler],
66-
)}
67-
checked={includeUserPages}
68-
/>
69-
{t('Include Subordinated Target Page', { target: '/user' })}
70-
</label>
71-
</div>
60+
{!isHidingUserPages && (
61+
<div className="me-3">
62+
<label className="form-label px-3 py-2 mb-0 d-flex align-items-center">
63+
<input
64+
className="me-2"
65+
type="checkbox"
66+
onChange={includeUserPagesChangeHandler}
67+
checked={includeUserPages}
68+
/>
69+
{t('Include Subordinated Target Page', { target: '/user' })}
70+
</label>
71+
</div>
72+
)}
7273
<div className="">
7374
<label className="form-label px-3 py-2 mb-0 d-flex align-items-center">
7475
<input

apps/app/src/features/search/client/components/SearchPage/SearchOptionModal/dynamic.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ type SearchOptionModalProps = {
66
isOpen: boolean;
77
includeUserPages: boolean;
88
includeTrashPages: boolean;
9+
isHidingUserPages: boolean;
910
onClose?: () => void;
1011
onIncludeUserPagesSwitched?: (isChecked: boolean) => void;
1112
onIncludeTrashPagesSwitched?: (isChecked: boolean) => void;

0 commit comments

Comments
 (0)