Make keepSelection the default behaviour

package.json

@@ -9,7 +9,8 @@
         "lint": "eslint packages/**/src/**/*.ts",
         "lint:fix": "eslint packages/**/src/**/*.ts --fix",
         "ci:dryrun": "npm run lint:fix && npm run build",
-        "prepublishOnly": "npm run lint && npm run build"
+        "prepublishOnly": "npm run lint && npm run build",
+        "postinstall": "lerna bootstrap"
     },
     "devDependencies": {
         "@typescript-eslint/eslint-plugin": "^4.26.1",

packages/react/demo/index.tsx

@@ -31,16 +31,11 @@ function SelectableArea({boxes, offset, className}: {
         });
     };
 
-    const onStop = ({selection}: SelectionEvent) => {
-        selection.keepSelection();
-    };
-
     return (
         <>
             <SelectionArea className={`container ${className}`}
                            onStart={onStart}
                            onMove={onMove}
-                           onStop={onStop}
                            selectables=".selectable">
                 {new Array(boxes).fill(0).map((_, index) => (
                     <div className={selected.has(index + offset) ? 'selected selectable' : 'selectable'}

packages/vanilla/README.md

@@ -86,7 +86,7 @@ const selection = new SelectionArea({
 
     // Specifies what should be done if already selected elements get selected again.
     //   invert: Invert selection for elements which were already selected
-    //   keep: Make stored elements (by keepSelection()) 'fix'
+    //   keep: Keep selected elements (use clearSelection() to remove those)
     //   drop: Remove stored elements after they have been touched
     overlap: 'invert',
 
@@ -149,8 +149,7 @@ selection.on('beforestart', evt => {
     console.log('move', evt);
 }).on('stop', evt => {
 
-    // The last event can be used to call functions like keepSelection() in case the user wants
-    // to select multiple elements.
+    // Do something with the selected elements.
     console.log('stop', evt);
 });
 ```

packages/vanilla/demo/index.ts

@@ -39,6 +39,4 @@ const selection = new SelectionArea({
         el.classList.remove('selected');
     }
 
-}).on('stop', () => {
-    selection.keepSelection();
 });

packages/vanilla/src/index.ts

@@ -519,6 +519,7 @@ export default class SelectionArea extends EventTarget<SelectionEvents> {
 
         // Hide selection area
         css(this._area, 'display', 'none');
+        this._keepSelection();
     }
 
     _updateElementSelection(): void {
@@ -588,30 +589,7 @@ export default class SelectionArea extends EventTarget<SelectionEvents> {
         });
     }
 
-    /**
-     * Manually triggers the start of a selection
-     * @param evt A MouseEvent / TouchEvent -like object
-     * @param silent If beforestart should be fired,
-     */
-    trigger(evt: MouseEvent | TouchEvent, silent = true): void {
-        this._onTapStart(evt, silent);
-    }
-
-    /**
-     * Can be used if during a selection elements have been added.
-     * Will update everything which can be selected.
-     */
-    resolveSelectables(): void {
-
-        // Resolve selectors
-        this._selectables = selectAll(this._options.selectables, this._options.document);
-    }
-
-    /**
-     * Saves the current selection for the next selecion.
-     * Allows multiple selections.
-     */
-    keepSelection(): void {
+    _keepSelection(): void {
         const {_options, _selection} = this;
         const {selected, changed, touched, stored} = _selection;
 
@@ -646,6 +624,25 @@ export default class SelectionArea extends EventTarget<SelectionEvents> {
         }
     }
 
+    /**
+     * Manually triggers the start of a selection
+     * @param evt A MouseEvent / TouchEvent -like object
+     * @param silent If beforestart should be fired,
+     */
+    trigger(evt: MouseEvent | TouchEvent, silent = true): void {
+        this._onTapStart(evt, silent);
+    }
+
+    /**
+     * Can be used if during a selection elements have been added.
+     * Will update everything which can be selected.
+     */
+    resolveSelectables(): void {
+
+        // Resolve selectors
+        this._selectables = selectAll(this._options.selectables, this._options.document);
+    }
+
     /**
      * Clear the elements which where saved by 'keepSelection()'.
      * @param store If the store should also get cleared