Fix: wrong types for canvas mouse events

[pieter-berkel]

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch @types/fabric@4.5.12 for the project I'm working on.

There were wrong types for some mouse events.

Here is the diff that solved my problem:

diff --git a/node_modules/@types/fabric/fabric-impl.d.ts b/node_modules/@types/fabric/fabric-impl.d.ts
index 107db96..1fc69dc 100755
--- a/node_modules/@types/fabric/fabric-impl.d.ts
+++ b/node_modules/@types/fabric/fabric-impl.d.ts
@@ -351,6 +351,20 @@ interface IObservable<T> {
+    off(
+        eventName: 
+            | 'mouse:up'
+            | 'mouse:down'
+            | 'mouse:move'
+            | 'mouse:up:before'
+            | 'mouse:down:before'
+            | 'mouse:move:before'
+            | 'mouse:dblclick'
+            | 'mouse:over'
+            | 'mouse:out', 
+        handler?: (e: IEvent<MouseEvent>) => void
+    ): T;
+    off(eventName: 'mouse:wheel', handler?: (e: IEvent<WheelEvent>) => void): T;
     off(eventName?: string | any, handler?: (e: IEvent) => void): T;
 
     /**

This issue body was partially generated by patch-package.

[burno1]

What exactly went wrong for you?

The only diff i can see is the type of event listener for generating the observable events