refactor(dev-only): disable note list keyboard nav when showing files table view
This commit is contained in:
@@ -182,6 +182,9 @@ const ContentListView = forwardRef<HTMLDivElement, Props>(
|
|||||||
}
|
}
|
||||||
}, [isFilesSmartView, filesController, createNewNote, toggleAppPane, application])
|
}, [isFilesSmartView, filesController, createNewNote, toggleAppPane, application])
|
||||||
|
|
||||||
|
const isFilesTableViewEnabled = featureTrunkEnabled(FeatureTrunkName.FilesTableView)
|
||||||
|
const shouldShowFilesTableView = isFilesTableViewEnabled && selectedTag?.uuid === SystemViewId.Files
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
const searchBarElement = document.getElementById(ElementIds.SearchBar)
|
const searchBarElement = document.getElementById(ElementIds.SearchBar)
|
||||||
/**
|
/**
|
||||||
@@ -204,6 +207,9 @@ const ContentListView = forwardRef<HTMLDivElement, Props>(
|
|||||||
if (searchBarElement === document.activeElement) {
|
if (searchBarElement === document.activeElement) {
|
||||||
searchBarElement?.blur()
|
searchBarElement?.blur()
|
||||||
}
|
}
|
||||||
|
if (shouldShowFilesTableView) {
|
||||||
|
return
|
||||||
|
}
|
||||||
selectNextItem()
|
selectNextItem()
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
@@ -211,6 +217,9 @@ const ContentListView = forwardRef<HTMLDivElement, Props>(
|
|||||||
command: PREVIOUS_LIST_ITEM_KEYBOARD_COMMAND,
|
command: PREVIOUS_LIST_ITEM_KEYBOARD_COMMAND,
|
||||||
element: document.body,
|
element: document.body,
|
||||||
onKeyDown: () => {
|
onKeyDown: () => {
|
||||||
|
if (shouldShowFilesTableView) {
|
||||||
|
return
|
||||||
|
}
|
||||||
selectPreviousItem()
|
selectPreviousItem()
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
@@ -252,6 +261,7 @@ const ContentListView = forwardRef<HTMLDivElement, Props>(
|
|||||||
selectNextItem,
|
selectNextItem,
|
||||||
selectPreviousItem,
|
selectPreviousItem,
|
||||||
selectionController,
|
selectionController,
|
||||||
|
shouldShowFilesTableView,
|
||||||
])
|
])
|
||||||
|
|
||||||
const shortcutForCreate = useMemo(
|
const shortcutForCreate = useMemo(
|
||||||
@@ -285,9 +295,6 @@ const ContentListView = forwardRef<HTMLDivElement, Props>(
|
|||||||
}
|
}
|
||||||
}, [selectedUuids, innerRef, isCurrentNoteTemplate, renderedItems, panes])
|
}, [selectedUuids, innerRef, isCurrentNoteTemplate, renderedItems, panes])
|
||||||
|
|
||||||
const isFilesTableViewEnabled = featureTrunkEnabled(FeatureTrunkName.FilesTableView)
|
|
||||||
const shouldShowFilesTableView = isFilesTableViewEnabled && selectedTag?.uuid === SystemViewId.Files
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div
|
<div
|
||||||
id={id}
|
id={id}
|
||||||
|
|||||||
Reference in New Issue
Block a user