Changeset 8d054ca in Klonkt

Timestamp:

06/27/2026 01:39:08 PM (2 months ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

923235e

Parents:

8951629

Message:

fix(ui): reply textareas no longer open huge (auto-resize skips hidden + caps height)

• views/shell.ejs — the global textarea auto-resize measured scrollHeight even on textareas hidden inside a closed <details> (the fedi owner-reply box), producing a bad height that stuck as inline style so the field opened huge. Now it skips hidden textareas (they size on focus once visible) and respects a CSS max-height.
• views/pages/post.ejs — give the comment/fedi-reply textarea a max-height (200px) so it opens at its min-height and scrolls past that instead of ballooning.

Location:

src/views

Files:

• pages/post.ejs (modified) (1 diff)
• shell.ejs (modified) (1 diff)

src/views/pages/post.ejs

@@ -383 +383 @@
   resize: vertical;
   min-height: 70px;
+  max-height: 200px;
   padding: 0.65rem 0.85rem;
   border: 1px solid var(--rule);

src/views/shell.ejs

@@ -684 +684 @@
   function autoSize(ta) {
     if (!ta || ta.tagName !== 'TEXTAREA') return;
-    ta.style.overflowY = 'hidden';
+    // Skip hidden textareas (e.g. inside a closed <details> or an unopened reply
+    // box): measuring scrollHeight there yields a bad height that sticks as inline
+    // style and makes the field open huge. They get sized on focus once visible.
+    if (ta.offsetParent === null && ta.offsetHeight === 0) return;
     ta.style.height = 'auto';
-    ta.style.height = ta.scrollHeight + 'px';
+    var maxH = parseFloat(getComputedStyle(ta).maxHeight);
+    var sh = ta.scrollHeight;
+    var h = (maxH && !isNaN(maxH)) ? Math.min(sh, maxH) : sh;   // respect a CSS max-height
+    ta.style.height = h + 'px';
+    ta.style.overflowY = sh > h ? 'auto' : 'hidden';
   }
   function sizeAll(root) {