fix: change snapshot order

Author: HarshKhandeparkar

src/renderers/RealDrawBoard/RealDrawBoard.ts

@@ -104,6 +104,8 @@ export class RealDrawBoard extends RealRenderer {
       this.canvas.addEventListener('mousemove', this._mouseMoveEventListener);
       this._isStroking = true;
 
+      if (this._currentSnapshotIndex < this._snapshots.length - 1 && this._maxSnapshots > 0) this._snapshots.splice(this._currentSnapshotIndex + 1); // Delete all redo snapshots
+
       this._startStroke(
         this._getMouseCoords(e),
         'mouse'
@@ -118,7 +120,14 @@ export class RealDrawBoard extends RealRenderer {
       this._isStroking = false;
 
       this._display(this.graphPixels);
+
       this.canvas.removeEventListener('mousemove', this._mouseMoveEventListener);
+
+      if (this._maxSnapshots > 0) this._snapshots[++this._currentSnapshotIndex] = this.getData(); // Take snapshot
+      if (this._snapshots.length > this._maxSnapshots) {
+        this._snapshots.shift();
+        this._currentSnapshotIndex--;
+      }
     }
   }
 
@@ -152,6 +161,8 @@ export class RealDrawBoard extends RealRenderer {
     for (let i = 0; i < e.touches.length; i++) {
       this._isStroking = true;
 
+      if (this._currentSnapshotIndex < this._snapshots.length - 1 && this._maxSnapshots > 0) this._snapshots.splice(this._currentSnapshotIndex + 1); // Delete all redo snapshots
+
       this._startStroke(
         this._getTouchCoords(e.touches.item(i)),
         e.touches.item(i).identifier.toString()
@@ -168,6 +179,12 @@ export class RealDrawBoard extends RealRenderer {
 
       this._isStroking = false;
     }
+
+    if (this._maxSnapshots > 0) this._snapshots[++this._currentSnapshotIndex] = this.getData(); // Take snapshot
+    if (this._snapshots.length > this._maxSnapshots) {
+      this._snapshots.shift();
+      this._currentSnapshotIndex--;
+    }
   }
 
   _touchMoveEventListener = (e: TouchEvent) => {

src/renderers/RealDrawBoard/tools/brush.ts

@@ -9,7 +9,6 @@ export function _startStroke(
   identifier: string
 ) {
   this._doPreview = false;
-  if (this._currentSnapshotIndex < this._snapshots.length - 1 && this._maxSnapshots > 0) this._snapshots.splice(this._currentSnapshotIndex + 1); // Delete all redo snapshots
   this._plot(coords[0], coords[1], this.brushSize, this.brushColor);
 
   this._lastCoords.set(identifier, coords);
@@ -24,12 +23,6 @@ export function _endStroke(
 
   this._lastCoords.delete(identifier);
 
-  if (this._maxSnapshots > 0) this._snapshots[++this._currentSnapshotIndex] = this.getData();
-  if (this._snapshots.length > this._maxSnapshots) {
-    this._snapshots.shift();
-    this._currentSnapshotIndex--;
-  }
-
   this._doPreview = true;
 }
 

src/renderers/RealDrawBoard/tools/eraser.ts

@@ -9,7 +9,6 @@ export function _startStroke(
   identifier: string
 ) {
   this._doPreview = false;
-  if (this._currentSnapshotIndex < this._snapshots.length - 1 && this._maxSnapshots > 0) this._snapshots.splice(this._currentSnapshotIndex + 1); // Delete all redo snapshots
   this._plot(coords[0], coords[1], this.eraserSize, this.bgColor);
 
   this._lastCoords.set(identifier, coords);
@@ -24,12 +23,6 @@ export function _endStroke(
   this._plot(endCoords[0], endCoords[1], this.eraserSize, this.bgColor);
 
   this._lastCoords.delete(identifier);
-
-  if (this._maxSnapshots > 0) this._snapshots[++this._currentSnapshotIndex] = this.getData();
-  if (this._snapshots.length > this._maxSnapshots) {
-    this._snapshots.shift();
-    this._currentSnapshotIndex--;
-  }
 }
 
 export function _doStroke(

src/renderers/RealDrawBoard/tools/line.ts

@@ -13,7 +13,6 @@ export function _startStroke(
   coords: [number, number],
   identifier: string
 ) {
-  if (this._currentSnapshotIndex < this._snapshots.length - 1 && this._maxSnapshots > 0) this._snapshots.splice(this._currentSnapshotIndex + 1); // Delete all redo snapshots
   this._plot(coords[0], coords[1], this.brushSize, this.brushColor);
 
   this._lastCoords.set(identifier, coords);