Skip to content

Fix #34 GUE window resize responsiveness#63

Draft
CoreyRDean wants to merge 1 commit into
developfrom
fix/34-gue-resize-projection
Draft

Fix #34 GUE window resize responsiveness#63
CoreyRDean wants to merge 1 commit into
developfrom
fix/34-gue-resize-projection

Conversation

@CoreyRDean
Copy link
Copy Markdown
Collaborator

Non-technical summary

This restores the GUE resize path so the editor stays aligned with the resized window instead of leaving the interface and pointer mapping out of sync. It matters now because issue #34 is a user-visible regression in a core editor workflow, and the fix stays narrowly focused on the shared resize seam rather than broad GUE cleanup.

Technical summary

  • add a small FUIResizeMetrics helper that computes the window aspect and GUI scale used by F-UI projection math
  • update src/Modules/F-UI.bb to rebuild the camera viewport and pivot transform during initialisation and whenever the app window size changes
  • cover the resize-metric contract with a focused Blitz test in src/Tests/Helpers/FUIResizeMetricsTest.bb
  • preserve the existing editor layout logic; this change only repairs the shared projection refresh that keeps input and rendered widgets aligned after a resize

Additional notes

  • Validation in this macOS environment is compile-oriented: git diff --check, compile-only -c -t for the new resize-metrics test, and compile-only -c for src/GUE.bb and src/Project Manager.bb
  • Full local test execution is still blocked by the current BlitzForge macOS arm64 stub runtime, and a full compile-only sweep also hits a pre-existing failure in src/Tests/Modules/AccountsServerTest.bb (New must call constructor)
  • The clean worktree could not build its own vendored BlitzForge compiler because the pinned submodule state still fails on macOS (windows.h dependency in BlitzForge sources), so validation used the already-built local blitzcc binary from the main repo checkout against this worktree's sources

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant