Hotfix: Prevent infinite event loop & remove visual noise

AllYourBot · May 21, 2024 · cfd8491 · cfd8491
1 parent 09a9e5c
commit cfd8491
Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/javascript/application.js b/app/javascript/application.js
@@ -18,7 +18,7 @@ setInterval(() => {
       if (elem) {
         elem.classList.remove('bg-red-400')
         elem.classList.remove('text-white')
-        elem.classList.add('text-red-500')
+        elem.classList.add('text-gray-200')
       }
     }
   }

diff --git a/app/javascript/stimulus/textarea_autogrow_controller.js b/app/javascript/stimulus/textarea_autogrow_controller.js
@@ -25,14 +25,14 @@ export default class extends Controller {
     window.removeEventListener('main-column-changed', this.throttledAutogrow)
   }
 
-  throttledAutogrow = throttle(() => this.autogrow(), 50)
+  throttledAutogrow = throttle((event) => { if (!event.detail.fromAutogrow) this.autogrow(event) }, 50)
   autogrow() {
     const prevHeight = getComputedStyle(this.element).height
     this.element.style.height = 'auto'
     const newHeight = `${this.element.scrollHeight+2}px` // scrollHeight is two less than computedHeight, this prevents jumps
     this.element.style.height = newHeight
 
-    if (prevHeight != newHeight) window.dispatchEvent(new CustomEvent('main-column-changed'))
+    if (prevHeight != newHeight) window.dispatchEvent(new CustomEvent('main-column-changed', { detail: { fromAutogrow: true } })) // TODO: prevents infinite event loop, why is this happening sometimes?
   }
 
   submitForm() {