fix: store bound synccomplete handler so dispose() actually removes it

TextView.dispose() called removeEventListener with .bind(this), which
creates a new function reference. The original listener was added with
a different .bind(this) call, so removeEventListener silently failed
and the listener was never removed.

Store the bound function as a class property and use the same reference
for both addEventListener and removeEventListener.

Fixes #178

Signed-off-by: Salman Muin Kayser Chishti <13schishti@gmail.com>

Author: salmanmkc

src/ui/components/TextView.ts

@@ -146,6 +146,7 @@ export class TextView extends View<TextViewEventMap> {
   /** The total number of lines after text wrapping. */
   lineCount = 0;
 
+  private _onSyncCompleteBound = this.onSyncComplete.bind(this);
   private _initializeTextCalled = false;
   private _text = 'TextView';
   set text(text) {
@@ -372,7 +373,7 @@ export class TextView extends View<TextViewEventMap> {
       this.textObj.addEventListener(
         // @ts-expect-error Missing type in Troika
         'synccomplete',
-        this.onSyncComplete.bind(this)
+        this._onSyncCompleteBound
       );
 
       if (this.imageOverlay) {
@@ -413,7 +414,7 @@ export class TextView extends View<TextViewEventMap> {
       this.textObj.removeEventListener(
         // @ts-expect-error Missing type in Troika
         'synccomplete',
-        this.onSyncComplete.bind(this)
+        this._onSyncCompleteBound
       );
     }
     super.dispose();