Changeset e22230e in Klonkt

Timestamp:

06/19/2026 12:38:30 PM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

de21a3b

Parents:

f3f43df

Message:

fix(editor): remove 350ms scroll guard + always type fullscreen on touch

• keepScroll: removed the 350ms scroll trap (itself caused a visible jump); now only sync + 1 frame restore, and nothing at all in fullscreen (page is locked there → field may scroll to caret).
• Auto-fullscreen on touch now also on beforeinput + pointerup (not just focus), so you can't accidentally type without fullscreen after leaving it.

Co-Authored-By: Claude <noreply@…>

File:

• src/views/pages/post-edit.ejs (modified) (3 diffs)

src/views/pages/post-edit.ejs

@@ -1096 +1096 @@
   }
   function keepScroll(fn) {
+    // In fullscreen staat de pagina vast (body overflow:hidden) en mag het veld
+    // gewoon naar de caret scrollen — geen pagina-sprong mogelijk, dus niets fixen.
+    const frame = document.querySelector('.pe-editor-frame');
+    if (frame && frame.classList.contains('pe-fs')) { fn(); return; }
     const wx = window.scrollX, wy = window.scrollY;
     const anc = scrollableAncestors(editor).map(function (n) { return [n, n.scrollTop, n.scrollLeft]; });
@@ -1104 +1108 @@
     fn();
     restore();
-    // Mobiel Chrome scrollt de caret ASYNCHROON in beeld (ná meerdere frames), ook
-    // bij een lege caret. Vang élke scroll de eerstvolgende ~350 ms af en zet 'm
-    // terug; daarna mag de gebruiker weer vrij scrollen.
-    const onScroll = function () { restore(); };
-    window.addEventListener('scroll', onScroll, true);
-    setTimeout(function () { window.removeEventListener('scroll', onScroll, true); }, 350);
+    requestAnimationFrame(restore);
   }
   function execCmd(cmd, arg) {
@@ -1259 +1258 @@
 
   // Op mobiel/tablet (touch-apparaat = pointer:coarse) gebeurt typen ALTIJD
-  // fullscreen: zodra het schrijfveld focus krijgt, gaan we automatisch fullscreen.
-  // Eruit kan met de knop (⤡) of Escape; tik je weer in het veld → opnieuw fullscreen.
+  // fullscreen. We gaan fullscreen zowel bij focus ALS bij de eerste invoer
+  // (beforeinput) — zo kun je ook na het verlaten van fullscreen niet per ongeluk
+  // zonder fullscreen doortypen. Eruit kan met de knop (⤡) of Escape.
   var isTouch = window.matchMedia && window.matchMedia('(pointer: coarse)').matches;
   if (isTouch) {
-    editor.addEventListener('focus', function () {
+    var enterFs = function () {
       if (editorFrame && !editorFrame.classList.contains('pe-fs')) toggleFullscreen();
-    });
+    };
+    editor.addEventListener('focus', enterFs);
+    editor.addEventListener('beforeinput', enterFs);
+    editor.addEventListener('pointerup', enterFs);
   }