fix(tools): fix TypeScript errors after migration to TS 5

christianliebel · May 16, 2024 · f2341e0 · f2341e0
1 parent 91f7f6c
commit f2341e0
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 23 deletions.
diff --git a/src/tools/brush.ts b/src/tools/brush.ts
@@ -74,18 +74,18 @@ export class BrushTool implements Tool {
     if (type === 'forward-diagonal') {
       if (posDiff && posDiff.y !== 0) {
         const xCorr = posDiff.y === -1 && posDiff.x === -1 ? x : x - 1;
-        this.drawForwardLine(xCorr, yCorr, diff, correction, size, context);
+        this.drawForwardLine(xCorr, yCorr, diff, correction, context);
       }
-      this.drawForwardLine(x, y, diff, correction, size, context);
+      this.drawForwardLine(x, y, diff, correction, context);
       return;
     }
 
     if (type === 'backward-diagonal') {
       if (posDiff && posDiff.y !== 0) {
         const xCorr = posDiff.y === -1 && posDiff.x === 1 ? x : x + 1;
-        this.drawBackwardLine(xCorr, yCorr, diff, correction, size, context);
+        this.drawBackwardLine(xCorr, yCorr, diff, correction, context);
       }
-      this.drawBackwardLine(x, y, diff, correction, size, context);
+      this.drawBackwardLine(x, y, diff, correction, context);
       return;
     }
 
@@ -107,7 +107,6 @@ export class BrushTool implements Tool {
     y: number,
     diff: number,
     correction: number,
-    size: number,
     context: CanvasRenderingContext2D,
   ): void {
     const start = { x: x - diff, y: y + diff + correction };
@@ -122,7 +121,6 @@ export class BrushTool implements Tool {
     y: number,
     diff: number,
     correction: number,
-    size: number,
     context: CanvasRenderingContext2D,
   ): void {
     const start = { x: x - diff, y: y - diff };

diff --git a/src/tools/ellipse.ts b/src/tools/ellipse.ts
@@ -19,15 +19,7 @@ export class EllipseTool implements Tool {
     color: ToolColor,
   ): void {
     if (canvas && previewContext) {
-      this.drawEllipse(
-        x,
-        y,
-        fillStyle,
-        color,
-        canvas,
-        previewContext,
-        previewContext,
-      );
+      this.drawEllipse(x, y, fillStyle, color, previewContext, previewContext);
     }
   }
 
@@ -38,7 +30,7 @@ export class EllipseTool implements Tool {
     color: ToolColor,
   ): void {
     if (canvas && context && previewContext) {
-      this.drawEllipse(x, y, fillStyle, color, canvas, context, previewContext);
+      this.drawEllipse(x, y, fillStyle, color, context, previewContext);
     }
   }
 
@@ -47,7 +39,6 @@ export class EllipseTool implements Tool {
     y: number,
     fillStyle: FillStyle,
     color: ToolColor,
-    canvas: HTMLCanvasElement,
     targetContext: CanvasRenderingContext2D,
     previewContext: CanvasRenderingContext2D,
   ): void {

diff --git a/src/tools/line.ts b/src/tools/line.ts
@@ -26,7 +26,7 @@ export class LineTool implements Tool {
     { previewContext, canvas, lineWidth }: DrawingContext,
   ): void {
     if (canvas && previewContext) {
-      this.drawLine(x, y, previewContext, previewContext, canvas, lineWidth);
+      this.drawLine(x, y, previewContext, previewContext, lineWidth);
     }
   }
 
@@ -36,7 +36,7 @@ export class LineTool implements Tool {
     { previewContext, context, canvas, lineWidth }: DrawingContext,
   ): void {
     if (previewContext && context && canvas) {
-      this.drawLine(x, y, context, previewContext, canvas, lineWidth);
+      this.drawLine(x, y, context, previewContext, lineWidth);
     }
   }
 
@@ -45,7 +45,6 @@ export class LineTool implements Tool {
     y: number,
     targetContext: CanvasRenderingContext2D,
     previewContext: CanvasRenderingContext2D,
-    canvas: HTMLCanvasElement,
     lineWidth: number,
   ): void {
     clearContext(previewContext);

diff --git a/src/tools/rectangle.ts b/src/tools/rectangle.ts
@@ -25,7 +25,6 @@ export class RectangleTool implements Tool {
         previewContext,
         fillStyle,
         lineWidth,
-        canvas,
         color,
       );
     }
@@ -45,7 +44,6 @@ export class RectangleTool implements Tool {
         previewContext,
         fillStyle,
         lineWidth,
-        canvas,
         color,
       );
     }
@@ -58,7 +56,6 @@ export class RectangleTool implements Tool {
     previewContext: CanvasRenderingContext2D,
     fillStyle: { fill: boolean; stroke: boolean },
     lineWidth: number,
-    canvas: HTMLCanvasElement,
     color: ToolColor,
   ): void {
     clearContext(previewContext);