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

chore: fix selection issue when deleting items on mobile [skip e2e]

Browse Source
This commit is contained in:
Aman Harwara
2023-10-27 22:12:28 +05:30
parent 66c7d183ab
commit abc8f371aa
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -266,8 +266,10 @@ export class ItemListController
this.disposers.push( this.disposers.push(
reaction( reaction(
() => this.selectedItemsCount, () => this.selectedItemsCount,
() => { (count, prevCount) => {
if (this.selectedItemsCount === 0) { const hasNoSelectedItem = count === 0
const onlyOneSelectedItemAfterChange = prevCount > count && count === 1
if (hasNoSelectedItem || onlyOneSelectedItemAfterChange) {
this.cancelMultipleSelection() this.cancelMultipleSelection()
} }
}, },
@@ -1193,9 +1195,6 @@ export class ItemListController
this.setSelectedUuids(this.selectedUuids) this.setSelectedUuids(this.selectedUuids)
this.lastSelectedItem = item this.lastSelectedItem = item
} }
if (this.selectedItemsCount === 1) {
this.cancelMultipleSelection()
}
} else { } else {
const shouldSelectNote = hasMoreThanOneSelected || !this.selectedUuids.has(uuid) const shouldSelectNote = hasMoreThanOneSelected || !this.selectedUuids.has(uuid)
if (shouldSelectNote && isAuthorizedForAccess) { if (shouldSelectNote && isAuthorizedForAccess) {