Skip to content

Capture the DPI once per pie build - #79

Merged
danielchalmers merged 1 commit into
mainfrom
claude/pie-dpi-once-per-build
Sep 21, 2026
Merged

danielchalmers merged 1 commit into
mainfrom
claude/pie-dpi-once-per-build

Conversation

@danielchalmers

@danielchalmers danielchalmers commented Sep 21, 2026 •

Copy link
Copy Markdown
Owner

Every snapped coordinate walked up to the presentation source through VisualTreeHelper.GetDpi, and a pie build snaps well over a hundred points across slices, hints, and labels. The DPI cannot change mid-build, so it is now read once at the start of CreatePieMenu.

Risk

  • A DPI change still rebuilds the pie through OnDpiChanged, which refreshes the capture, so the cached value can never go stale across a monitor move.

@danielchalmers
danielchalmers added this pull request to stack #81 September 21, 2026 02:19
@danielchalmers
danielchalmers force-pushed the claude/pie-dpi-once-per-build branch from 690559d to 1e76aa3 Compare September 21, 2026 02:49
@danielchalmers
danielchalmers removed this pull request from stack #81 September 21, 2026 02:50
@danielchalmers
danielchalmers added this pull request to stack #82 September 21, 2026 02:51
@danielchalmers
danielchalmers force-pushed the claude/pie-dpi-once-per-build branch from 1e76aa3 to 8207895 Compare September 21, 2026 02:53
Base automatically changed from claude/pie-idle-prebuild to main September 21, 2026 02:54
Every snapped coordinate walked up to the presentation source through
VisualTreeHelper.GetDpi, and a pie build snaps well over a hundred
points across slices, hints, and labels. The DPI cannot change mid-build,
so it is now read once at the start of CreatePieMenu. A DPI change still
rebuilds the pie through OnDpiChanged, which refreshes the capture.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AQy32pEEDt7qxvE76cJk6G
@danielchalmers
danielchalmers force-pushed the claude/pie-dpi-once-per-build branch from 8207895 to fd7596f Compare September 21, 2026 02:54
@danielchalmers
danielchalmers merged commit 4e48de9 into main Sep 21, 2026
1 check passed
@danielchalmers
danielchalmers deleted the claude/pie-dpi-once-per-build branch September 21, 2026 02:54
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