fix: scrolling! (#28)

Author: eirikb

lib/widgets/list.ts

@@ -527,11 +527,9 @@ class List extends ScrollableBox {
     }
   }
 
-  // Basic scrollTo implementation for list functionality
+  // Use parent ScrollableBox scrollTo implementation
   scrollTo(offset: number, always?: boolean): any {
-    // For now, just a stub implementation
-    // This should be replaced with proper scrolling logic
-    return;
+    return super.scrollTo(offset, always);
   }
 
   select = (index: any): void => {

lib/widgets/terminal.ts

@@ -115,6 +115,10 @@ class Terminal extends Box {
     }
 
     this.pty.onData(() => {
+      // Auto-scroll to bottom when new content arrives
+      if (this.term && typeof this.term.scrollToBottom === 'function') {
+        this.term.scrollToBottom();
+      }
       setTimeout(() => this.screen.render(), 16);
     });
 
@@ -186,11 +190,13 @@ class Terminal extends Box {
     if (xi >= xl || yi >= yl) return ret;
 
     const buf = this.term.buffer.active;
+    const viewportY = buf.viewportY || 0; // Get the current scroll position
+
     for (var y = yi; y < yl; y++) {
       var line = this.screen.lines[y];
       if (!line) continue;
 
-      var bufferY = y - yi;
+      var bufferY = y - yi + viewportY; // Account for scroll position
       var bufferLine = buf.getLine(bufferY);
       if (!bufferLine) continue;
 
@@ -264,6 +270,30 @@ class Terminal extends Box {
     }
   }
 
+  scrollTo(line: number): void {
+    if (this.term && typeof this.term.scrollToLine === 'function') {
+      this.term.scrollToLine(line);
+    }
+  }
+
+  scroll(offset: number): void {
+    if (this.term && typeof this.term.scrollLines === 'function') {
+      this.term.scrollLines(offset);
+    }
+  }
+
+  scrollToTop(): void {
+    if (this.term && typeof this.term.scrollToTop === 'function') {
+      this.term.scrollToTop();
+    }
+  }
+
+  scrollToBottom(): void {
+    if (this.term && typeof this.term.scrollToBottom === 'function') {
+      this.term.scrollToBottom();
+    }
+  }
+
   kill() {
     if (this.pty) this.pty.kill();
     if (this.term) this.term.dispose();