From abfd8e098399ce2a6491af6bff4ae553dfee56ec Mon Sep 17 00:00:00 2001
From: Steven Hugg <hugg@fasterlight.com>
Date: Mon, 28 Sep 2026 22:55:05 +0200
Subject: [PATCH] vcs-core: 223 lines, aspect ratio

---
 src/machine/vcs.ts | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/machine/vcs.ts b/src/machine/vcs.ts
index 97b98b74..c3241cee 100644
--- a/src/machine/vcs.ts
+++ b/src/machine/vcs.ts
@@ -194,7 +194,10 @@ export class JavatariMachine implements FrameBased, RasterFrameBased, VideoSourc
   // VIDEO
 
   getVideoParams(): VideoParams {
-    return { width: VISIBLE_WIDTH, height: VISIBLE_HEIGHT, aspect: VISIBLE_WIDTH * 2 / VISIBLE_HEIGHT, overscan: true };
+    // a 2600's picture is 4:3 whatever the pixel grid happens to be -- with
+    // overscan showing 223 lines, the pixels are about 1.86:1, so the ratio
+    // has to be stated rather than derived from the width and height
+    return { width: VISIBLE_WIDTH, height: VISIBLE_HEIGHT, aspect: 4 / 3, overscan: true };
   }
   connectVideo(pixels: Uint32Array) {
     this.pixels = pixels;