Skip to content

[codex] Fix Windows screenshot DPI and display capture#503

Open
fffonion wants to merge 1 commit into
openai:mainfrom
fffonion:codex/fix-windows-screenshot-dpi
Open

[codex] Fix Windows screenshot DPI and display capture#503
fffonion wants to merge 1 commit into
openai:mainfrom
fffonion:codex/fix-windows-screenshot-dpi

Conversation

@fffonion

@fffonion fffonion commented Jun 22, 2026

Copy link
Copy Markdown

Summary

Fixes Windows screenshot capture behavior in the curated screenshot skill.

This commit is drafted and created by Codex.

Changes

  • Enables per-monitor DPI awareness before reading Windows display and window rectangles.
  • Uses DWM extended frame bounds for window captures, with GetWindowRect as a fallback.
  • Adds milliseconds to generated filenames to reduce same-second collisions.
  • Updates the skill documentation for the new Windows behavior.

Validation

  • Ran work/test_screenshot_windows.ps1 against the curated skill helper on a mixed-DPI three-display Windows setup.
  • Verified default capture size matched the virtual desktop.
  • Verified a Codex window capture matched DWM visible frame bounds.

@fffonion fffonion force-pushed the codex/fix-windows-screenshot-dpi branch from fab223c to 8a85cd9 Compare June 22, 2026 17:04
@fffonion fffonion force-pushed the codex/fix-windows-screenshot-dpi branch from 8a85cd9 to c31ab95 Compare June 22, 2026 17:07
@fffonion fffonion marked this pull request as ready for review June 22, 2026 17:08
@fffonion fffonion requested a review from a team June 22, 2026 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant