rmorawski/standardnotes-app-web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: dont autofocus menu items on mobile

Browse Source
This commit is contained in:
Aman Harwara
2023-05-12 22:00:27 +05:30
parent 2e11c8a7d1
commit aa1f7d976e
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
packages/web/src/javascripts/Components/Menu/Menu.tsx
View File

@@ -10,6 +10,7 @@ import {
import { KeyboardKey } from '@standardnotes/ui-services' import { KeyboardKey } from '@standardnotes/ui-services'
import { useListKeyboardNavigation } from '@/Hooks/useListKeyboardNavigation' import { useListKeyboardNavigation } from '@/Hooks/useListKeyboardNavigation'
import { mergeRefs } from '@/Hooks/mergeRefs' import { mergeRefs } from '@/Hooks/mergeRefs'
import { MutuallyExclusiveMediaQueryBreakpoints, useMediaQuery } from '@/Hooks/useMediaQuery'
type MenuProps = { type MenuProps = {
className?: string className?: string
@@ -51,7 +52,13 @@ const Menu = forwardRef(
[closeMenu, onKeyDown], [closeMenu, onKeyDown],
) )
const { setInitialFocus } = useListKeyboardNavigation(menuElementRef, initialFocus, shouldAutoFocus) const isMobileScreen = useMediaQuery(MutuallyExclusiveMediaQueryBreakpoints.sm)
const { setInitialFocus } = useListKeyboardNavigation(
menuElementRef,
initialFocus,
isMobileScreen ? false : shouldAutoFocus,
)
useImperativeHandle(forwardedRef, () => ({ useImperativeHandle(forwardedRef, () => ({
focus: () => { focus: () => {