browse: set default viewport to 1280x720 (16:9 widescreen)
Improve browser screenshot quality by setting a modern default viewport
size during browser initialization. This replaces the previous system-
dependent default with a consistent 1280x720 resolution that provides:
- Better readability and content coverage in screenshots
- Modern 16:9 aspect ratio matching typical displays
- Consistent testing environment across all users
Implementation:
- Add EmulateViewport call during browser startup in browse.go
- Set viewport immediately after browser initialization
- Add comprehensive test (TestDefaultViewportSize) to verify functionality
The test confirms the viewport is correctly set to 1280x720 on browser
startup and validates the JavaScript window dimensions match expectations.
This addresses screenshot quality issues with small default viewport sizes
and provides a better default experience for browser automation.
Co-Authored-By: sketch <hello@sketch.dev>
Change-ID: s98815cfd1eaca88dk
diff --git a/claudetool/browse/browse.go b/claudetool/browse/browse.go
index fdc83c6..9c2cca1 100644
--- a/claudetool/browse/browse.go
+++ b/claudetool/browse/browse.go
@@ -92,6 +92,13 @@
b.initErr = fmt.Errorf("failed to start browser (please apt get chromium or equivalent): %w", err)
return
}
+
+ // Set default viewport size to 1280x720 (16:9 widescreen)
+ if err := chromedp.Run(browserCtx, chromedp.EmulateViewport(1280, 720)); err != nil {
+ b.initErr = fmt.Errorf("failed to set default viewport: %w", err)
+ return
+ }
+
b.initialized = true
})