Fix Block selection via CMD+A

docs/CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - `Refactoring` — Constants of tools settings separated by internal and external to correspond API
 - `Refactoring` — Created universal Flipper class that responses for navigation by keyboard inside of any Toolbars
+- `Fix` — First CMD+A on block with now uses default behaviour. Fixed problem with second CMD+A after selection clearing [#827](https://github.com/codex-team/editor.js/issues/827)
 
 ### 2.15
 

src/components/modules/blockEvents.ts

@@ -166,6 +166,12 @@ export default class BlockEvents extends Module {
    * @param {MouseEvent} event
    */
   public mouseDown(event: MouseEvent): void {
+    /**
+     * Each mouse down on Block must disable selectAll state
+     */
+    if (!SelectionUtils.isCollapsed) {
+      this.Editor.BlockSelection.clearSelection(event);
+    }
     this.Editor.CrossBlockSelection.watchSelection(event);
   }
 

src/components/modules/blockSelection.ts

@@ -263,8 +263,8 @@ export default class BlockSelection extends Module {
     this.Editor.RectangleSelection.clearSelection();
 
     /** allow default selection on native inputs */
-    if ($.isNativeInput(event.target) && !this.nativeInputSelected) {
-      this.nativeInputSelected = true;
+    if ($.isNativeInput(event.target) && !this.readyToBlockSelection) {
+      this.readyToBlockSelection = true;
       return;
     }
 
@@ -280,30 +280,35 @@ export default class BlockSelection extends Module {
       return;
     }
 
-    if (this.needToSelectAll) {
-      /** Prevent default selection */
-      event.preventDefault();
+    if (inputs.length === 1 && !this.needToSelectAll) {
+      this.needToSelectAll = true;
+      return;
+    }
 
+    if (this.needToSelectAll) {
       /**
-       * Save selection
-       * Will be restored when closeSelection fired
+       * Prevent default selection
        */
-      this.selection.save();
+      event.preventDefault();
+
+      this.selectAllBlocks();
 
       /**
-       * Remove Ranges from Selection
+       * Disable any selection after all Blocks selected
        */
-      SelectionUtils.get()
-        .removeAllRanges();
-
-      this.selectAllBlocks();
       this.needToSelectAll = false;
+      this.readyToBlockSelection = false;
 
       /**
        * Close ConversionToolbar when all Blocks selected
        */
       this.Editor.ConversionToolbar.close();
-    } else {
+    } else if (this.readyToBlockSelection) {
+      this.selectBlockByIndex();
+
+      /**
+       * Enable all Blocks selection if current Block is selected
+       */
       this.needToSelectAll = true;
     }
   }
@@ -313,6 +318,18 @@ export default class BlockSelection extends Module {
    * Each Block has selected setter that makes Block copyable
    */
   private selectAllBlocks() {
+    /**
+     * Save selection
+     * Will be restored when closeSelection fired
+     */
+    this.selection.save();
+
+    /**
+     * Remove Ranges from Selection
+     */
+    SelectionUtils.get()
+      .removeAllRanges();
+
     this.allBlocksSelected = true;
   }
 }