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

Merge pull request #2985 from standardnotes/fix/multiple-selection-issues

Browse Source 
fix: multiple selection issues
This commit is contained in:
Antonella Sgarlatta
2026-02-24 13:02:32 -03:00
committed by GitHub
parent f4c64b8db2 99c7cb92b5
commit ec7d3970d0
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
packages/web/src/javascripts/Controllers/ItemList/ItemListController.ts
View File

@@ -269,10 +269,9 @@ export class ItemListController
this.disposers.push( this.disposers.push(
reaction( reaction(
() => this.selectedItemsCount, () => this.selectedItemsCount,
(count, prevCount) => { (count) => {
const hasNoSelectedItem = count === 0 const hasNoSelectedItem = count === 0
const onlyOneSelectedItemAfterChange = prevCount > count && count === 1 if (hasNoSelectedItem) {
if (hasNoSelectedItem || onlyOneSelectedItemAfterChange) {
this.cancelMultipleSelection() this.cancelMultipleSelection()
} }
}, },
@@ -1187,7 +1186,7 @@ export class ItemListController
if (userTriggered && hasShift && !isMobileScreen()) { if (userTriggered && hasShift && !isMobileScreen()) {
await this.selectItemsRange({ selectedItem: item }) await this.selectItemsRange({ selectedItem: item })
} else if (userTriggered && this.isMultipleSelectionMode) { } else if (userTriggered && this.isMultipleSelectionMode) {
if (this.selectedUuids.has(uuid) && hasMoreThanOneSelected) { if (this.selectedUuids.has(uuid)) {
this.removeSelectedItem(uuid) this.removeSelectedItem(uuid)
} else if (isAuthorizedForAccess) { } else if (isAuthorizedForAccess) {
this.selectedUuids.add(uuid) this.selectedUuids.add(uuid)