Improve transitioning from batch mode

Having "search" mode as default transition from "batch" mode was not
ideal because if a table is sortable and does not have a search field,
leaving "batch" mode would turn on "search" mode, mode which disables
sorting.

Instead, let's just remember which mode we were on before going into
"batch", and then restore it.

Author: chaimann

admin/app/components/solidus_admin/ui/table/component.js

@@ -69,27 +69,21 @@ export default class extends Controller {
 
   selectRow(event) {
     if (this.checkboxTargets.some((checkbox) => checkbox.checked)) {
+      this.previousMode = this.modeValue;
       this.modeValue = "batch"
-    } else if (this.hasSearchFieldTarget && this.searchFieldTarget.value !== "") {
-      this.modeValue = "search"
-    } else if (this.hasScopesToolbarTarget) {
-      this.modeValue = "scopes"
     } else {
-      this.modeValue = "search"
+      this.modeValue = this.previousMode;
     }
 
     this.render()
   }
 
   selectAllRows(event) {
     if (event.target.checked) {
+      this.previousMode = this.modeValue;
       this.modeValue = "batch"
-    } else if (this.hasSearchFieldTarget && this.searchFieldTarget.value !== "") {
-      this.modeValue = "search"
-    } else if (this.hasScopesToolbarTarget) {
-      this.modeValue = "scopes"
     } else {
-      this.modeValue = "search"
+      this.modeValue = this.previousMode;
     }
 
     this.checkboxTargets.forEach((checkbox) => (checkbox.checked = event.target.checked))