@cplieger/web-terminal-ui 6.1.4 → 6.1.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/web-terminal-ui",
-  "version": "6.1.4",
+  "version": "6.1.5",
   "type": "module",
   "description": "Reference touch-first browser UI for @cplieger/web-terminal-engine: a composable kernel + feature modules (tabs, activity monitor, mobile toolbar, context menu, IME, predictive echo, viewport handling) and CSS. One createTerminal() call.",
   "license": "MPL-2.0",
@@ -52,7 +52,7 @@
     "@stryker-mutator/vitest-runner": "10.0.0",
     "@types/node": "26.2.0",
     "@typescript/native": "npm:typescript@7.0.2",
-    "@vitest/coverage-v8": "4.1.10",
+    "@vitest/coverage-v8": "4.1.11",
     "eslint": "10.8.1",
     "fast-check": "4.9.0",
     "happy-dom": "20.11.2",
@@ -63,7 +63,7 @@
     "stylelint-config-standard": "40.0.0",
     "typescript": "npm:@typescript/typescript6@6.0.2",
     "typescript-eslint": "8.67.0",
-    "vitest": "4.1.10"
+    "vitest": "4.1.11"
   },
   "publishConfig": {
     "access": "public"

package/src/features/scroll-to-bottom.ts

@@ -47,7 +47,16 @@ export function scrollToBottom(): TerminalFeature {
           ctx.scroll.scrollToBottom();
           return;
         }
-        surface.scrollTo({ top: surface.scrollHeight, behavior: "smooth" });
+        // Target the MAXIMUM offset, not scrollHeight. An over-scroll target is
+        // one clientHeight past the end and needs the container to clamp it,
+        // which is the assumption the engine's own bottom writes stopped making:
+        // clamping an offset the content shrank out from under is an
+        // implementation behaviour and WebKit does not do it. The line above
+        // already computes the maximum, so this is the same arithmetic said once.
+        surface.scrollTo({
+          top: surface.scrollHeight - surface.clientHeight,
+          behavior: "smooth",
+        });
       };
 
       // pointerdown (like the toolbar keys) so touch devices get the jump on the

package/src/kernel/kernel.ts

@@ -811,7 +811,12 @@ function buildTerminal(
       case "scroll-down": {
         ev.preventDefault();
         const h = termWrap.clientHeight;
-        termWrap.scrollTop = Math.min(termWrap.scrollHeight, termWrap.scrollTop + h);
+        // The ceiling is the maximum offset, not scrollHeight: the latter is one
+        // clientHeight past the end, so a page-down near the bottom handed the
+        // container an out-of-range offset and relied on it to clamp. Same
+        // reasoning as the engine's bottom writes.
+        const max = Math.max(0, termWrap.scrollHeight - h);
+        termWrap.scrollTop = Math.min(max, termWrap.scrollTop + h);
         return true;
       }
       case "ignore":