Add slim custom title bar and status path display#52
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
WalkthroughThis PR implements a custom frameless window UI for version 1.6.1, introducing a slim custom title bar with hover-revealed menus, window control buttons, a zoom indicator, and directory path display in the status bar. It exposes zoom scale properties from the image rotator component and updates version identifiers. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3582be1c5f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| minimumWidth: 800 | ||
| minimumHeight: 500 | ||
| title: "FastStack - " + (uiState ? uiState.currentDirectory : "Loading...") | ||
| flags: Qt.FramelessWindowHint | Qt.Window | Qt.WindowMinMaxButtonsHint |
There was a problem hiding this comment.
Restore maximized loupe windows after frameless mode
After introducing Qt.FramelessWindowHint, maximized windows on Windows can report Window.FullScreen (as your new isMaximized comment notes), but exitFullScreenLoupe() still only restores maximized state when savedWindowGeometry.visibility === Window.Maximized. In the common flow “maximize window -> enter loupe fullscreen -> exit loupe fullscreen”, the saved state is FullScreen, so the app now returns to a normal window instead of the previous maximized state.
Useful? React with 👍 / 👎.
Summary by CodeRabbit
Release Notes - Version 1.6.1