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

chore: potential fix for unintentional multiple selection after editing on ios [skip e2e]

Browse Source
This commit is contained in:
Aman Harwara
2023-11-09 18:49:48 +05:30
parent 1bf3a99911
commit 9d532e83e1
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
packages/web/src/javascripts/Components/ContentListView/NoteListItem.tsx
View File

@@ -78,7 +78,8 @@ const NoteListItem: FunctionComponent<DisplayableListItemProps<SNNote>> = ({
const onClick = useCallback( const onClick = useCallback(
(event: MouseEvent) => { (event: MouseEvent) => {
if ((event.ctrlKey || event.metaKey) && !application.itemListController.isMultipleSelectionMode) { const hasMultiSelectionModifierKey = !isMobileScreen && (event.ctrlKey || event.metaKey)
if (hasMultiSelectionModifierKey && !application.itemListController.isMultipleSelectionMode) {
application.itemListController.enableMultipleSelectionMode() application.itemListController.enableMultipleSelectionMode()
} }
if (selected && !application.itemListController.isMultipleSelectionMode) { if (selected && !application.itemListController.isMultipleSelectionMode) {
@@ -87,7 +88,7 @@ const NoteListItem: FunctionComponent<DisplayableListItemProps<SNNote>> = ({
} }
onSelect(item, true).catch(console.error) onSelect(item, true).catch(console.error)
}, },
[application.itemListController, item, onSelect, selected], [application.itemListController, isMobileScreen, item, onSelect, selected],
) )
useContextMenuEvent(listItemRef, handleContextMenuEvent) useContextMenuEvent(listItemRef, handleContextMenuEvent)