@happy-nut/monacori 0.1.15 → 0.1.17

package/dist/viewer.client.js

@@ -509,6 +509,15 @@ function scheduleDiffScroll(row) {
 // reflow-forcing scrollIntoView can run, so collapse them to one (latest element) per animation frame and
 // use block:nearest (no per-row center-jump). Shared by the tree, source caret, and diff caret.
 var pendingScrollEl = null, scrollElRaf = 0;
+// Reveal `el` by keeping it ~fraction of the way down its scroller, scrolling minimally on EVERY move so the
+// view follows the caret CONTINUOUSLY. scrollIntoView('nearest') instead leaves the view still until the
+// caret reaches the edge and then jumps — stuttering ~every viewport while an arrow key is held.
+function revealAt(el, scroller, fraction) {
+  if (!el) return;
+  if (!scroller || !scroller.clientHeight) { try { el.scrollIntoView({ block: 'nearest', inline: 'nearest' }); } catch (x) {} return; }
+  var off = el.getBoundingClientRect().top - scroller.getBoundingClientRect().top;
+  scroller.scrollTop += off - scroller.clientHeight * fraction;
+}
 function scheduleScrollIntoView(el) {
   pendingScrollEl = el || null;
   if (scrollElRaf) return;
@@ -897,7 +906,7 @@ function scheduleTreeFocus() {
     if (treeFocusIndex < 0 || treeFocusIndex >= rows.length) return;
     const el = rows[treeFocusIndex];
     document.querySelectorAll('.tree-focus').forEach((e) => { if (e !== el) e.classList.remove('tree-focus'); });
-    if (el) { el.classList.add('tree-focus'); el.scrollIntoView({ block: 'nearest', inline: 'nearest' }); }
+    if (el) { el.classList.add('tree-focus'); revealAt(el, document.querySelector('.sidebar-scroll'), 0.42); }
   });
 }
 
@@ -1008,6 +1017,15 @@ function handleTreeKey(event) {
   return false;
 }
 
+// d2h-file-side-diff is a HORIZONTAL scrollport (overflow-x:auto), so the browser swallows VERTICAL wheel
+// there instead of bubbling it to #diff2html-container — the diff wouldn't scroll by mouse wheel at all.
+// Redirect vertical wheel to the container; leave horizontal wheel (deltaX) for the side's own h-scroll.
+(function () {
+  var dsc = document.getElementById('diff2html-container');
+  if (dsc) dsc.addEventListener('wheel', function (e) {
+    if (Math.abs(e.deltaY) >= Math.abs(e.deltaX) && e.deltaY !== 0) { dsc.scrollTop += e.deltaY; e.preventDefault(); }
+  }, { passive: false });
+})();
 document.addEventListener('keydown', (event) => {
   if (!quickOpen?.classList.contains('hidden')) {
     if (handleQuickOpenKey(event)) return;
@@ -1114,6 +1132,13 @@ document.addEventListener('keydown', (event) => {
     }
   }
 
+  // PageUp/Down scroll the diff/source view. There's no focusable scroller (the diff caret is a JS cursor),
+  // and d2h-file-side-diff's horizontal scrollport even swallows vertical wheel, so handle paging explicitly.
+  // Only when the tree isn't focused — the tree pages itself in handleTreeKey below.
+  if (treeFocusIndex < 0 && (event.key === 'PageDown' || event.key === 'PageUp') && !event.metaKey && !event.ctrlKey && !event.altKey) {
+    var psc = isDiffViewVisible() ? document.getElementById('diff2html-container') : (isSourceViewerVisible() ? document.getElementById('source-body') : null);
+    if (psc) { event.preventDefault(); psc.scrollTop += (event.key === 'PageDown' ? 0.9 : -0.9) * psc.clientHeight; return; }
+  }
   if (treeFocusIndex >= 0 && handleTreeKey(event)) return;
   if (treeFocusIndex < 0 && !event.metaKey && !event.ctrlKey && !event.altKey && isSourceViewerVisible() && handleSourceCaretKey(event)) return;
   if (treeFocusIndex < 0 && !event.metaKey && !event.ctrlKey && !event.altKey && isDiffViewVisible() && handleDiffCaretKey(event)) return;
@@ -1567,7 +1592,7 @@ function scheduleDiffReveal(wrapper, side, ri) {
     applyDiffSelection();
     if (!t) return;
     var row = diffRowAt(t.wrapper, t.side, t.ri);
-    if (row && row.scrollIntoView) { try { row.scrollIntoView({ block: 'nearest', inline: 'nearest' }); } catch (x) {} }
+    revealAt(row, document.getElementById('diff2html-container'), 0.42);
   });
 }
 function navEntryOf(kind) {
@@ -3302,7 +3327,7 @@ function scheduleSourceReveal(prev) {
     var lines = f.content.split(/\r?\n/);
     updateSourceCaret(p, lines, f.language || 'text');
     var cl = document.querySelector('.source-row.cursor-line');
-    if (cl && cl.scrollIntoView) { try { cl.scrollIntoView({ block: 'nearest', inline: 'nearest' }); } catch (x) {} }
+    revealAt(cl, document.getElementById('source-body'), 0.42);
   });
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happy-nut/monacori",
-  "version": "0.1.15",
+  "version": "0.1.17",
   "description": "Validation control plane for AI-generated code changes.",
   "type": "module",
   "repository": {