Skip to content

Refactor window handling, improve theme support, and update dependencies#1807

Draft
ItsEeleeya wants to merge 23 commits into
CapSoftware:mainfrom
ItsEeleeya:next-base-improvements
Draft

Refactor window handling, improve theme support, and update dependencies#1807
ItsEeleeya wants to merge 23 commits into
CapSoftware:mainfrom
ItsEeleeya:next-base-improvements

Conversation

@ItsEeleeya
Copy link
Copy Markdown
Contributor

This PR refactors a few things with small improvements here and there as a base for the upcoming PRs.

The header height on editor windows now matches that of windows with Toolbars on macOS 26.
Renamed ShowCapWindow to CapWindow.
Moved display related helper logic to src-tauri/src/display_utils.rs of out windows.rs
Renamed AppTheme to Appearance (with serde fallback).
Re-implemented the way windows are auto shown when ready.
Fixed an issue when changing the theme back to "system" didn't properly take effect until a reload.
Use of the new data-tauri-drag-region="deep" to prevent having to scatter it across elements.
Main window is no longer a NSPanel. It still appears on the same level.
Removed the fake traffic lights.
Removed the custom AppDelegate since Tauri can position the traffic lights.
Brought back onMouseDown={showCropOptionsMenu} within the Editor's crop section. This is the proper behavior of native menus. onClick is used as fallback for triggering it with a keyboard or touch.
Bumped objc2-app-kit version.
Fake window no longer sets focus. On macOS, coupled with the NSWindowStyleMask::NonactivatingPanel style mask it no longer steals focus from the active app even when clicked.
(It currently does not show hover effects)

ItsEeleeya added 23 commits May 7, 2026 15:37
The header height now matches that of windows with Toolbars on macOS 26
The header height now matches that of windows with Toolbars on macOS 26
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