Skip to content

fix(desktop): launch at sane default window size #1489

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
tellaho wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(desktop): launch at sane default window size #1489

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 7 additions & 9 deletions desktop/src-tauri/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,16 +84,14 @@ pub fn run() {
.plugin(tauri_plugin_opener::init())
.plugin(
tauri_plugin_window_state::Builder::default()
// The main window should always launch edge-to-edge in the
// available desktop area. Do not let stale saved geometry or
// fullscreen state override the maximized launch config.
// Restore the user's window size/position (and maximized
// state) across launches. First launch falls back to the
// centered default bounds in tauri.conf.json instead of
// opening edge-to-edge. Visibility stays managed by the app
// (window is revealed once React mounts) and fullscreen is
// never restored — relaunching into fullscreen is jarring.
.with_state_flags(
StateFlags::all()
& !(StateFlags::VISIBLE
| StateFlags::POSITION
| StateFlags::SIZE
| StateFlags::MAXIMIZED
| StateFlags::FULLSCREEN),
StateFlags::all() & !(StateFlags::VISIBLE | StateFlags::FULLSCREEN),
)
.build(),
)
Expand Down
6 changes: 3 additions & 3 deletions desktop/src-tauri/tauri.conf.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@
"windows": [
{
"title": "",
"width": 800,
"height": 600,
"maximized": true,
"width": 1280,
"height": 800,
"center": true,

@wesbillman wesbillman Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this might have been maximized by @klopez4212 maybe for onboarding?

@tellaho tellaho Jul 3, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@klopez4212 ah yep, sent your way last night kenny. any thoughts before I swap our window sizing back to more typical behavior?

@klopez4212 klopez4212 Jul 3, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah yeah go for it..I was just going for a bigger window in general. If possible to remember the last setting would be good but otherwise fixed is 👍

"visible": false,
"titleBarStyle": "Overlay",
"hiddenTitle": true,
Expand Down
6 changes: 5 additions & 1 deletion desktop/src/app/App.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,11 @@ export function App() {
useReloadShortcut();

useLayoutEffect(() => {
void getCurrentWindow().show();
// The window starts hidden (visible: false in tauri.conf.json) so users
// never see an unstyled flash. Reveal it once React mounts, and focus it
// explicitly — show() alone can leave the window behind others on launch.
const appWindow = getCurrentWindow();
void appWindow.show().then(() => appWindow.setFocus());
}, []);

const [sharedIdentity, setSharedIdentity] = useState<boolean | null>(null);
Expand Down
Loading