[WIP] Update Tauri v2 window-state persistence behavior on Windows - #5
Draft
crazyrob425 with Copilot wants to merge 1 commit into
Draft
[WIP] Update Tauri v2 window-state persistence behavior on Windows#5crazyrob425 with Copilot wants to merge 1 commit into
crazyrob425 with Copilot wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.
Original prompt
Update crazyrob425/KDP-E-Book-Generator (base main) Tauri v2 wrapper window-state persistence behavior on Windows to restore to the same monitor when possible; otherwise restore to a monitor that contains the saved top-left corner, with clamping to the monitor work area. Persist size/position/maximized state to a JSON file under AppData along with other app data. Use minimal Rust changes with Tauri v2 APIs, debounce saves, and update README to document behavior and AppData paths. Open a PR targeting main.
The following is the prior conversation context from the user's chat exploration (may be truncated):
User: @copilot Accepted Confirmation: Are you sure?
Assistant: Yes—forking is a good workaround if you can’t (or don’t want to) fight the permission issue on the original repo.
What I can do vs. what I can’t do
blacklistedrob-prog/KDP-E-Book-Generatorfrom here right now because this session doesn’t have the necessary write privileges on that repo, and forking itself may also be blocked depending on what GitHub allows the integration to do in your account.If forking is available on your side, this is the fastest path
You do one click:
blacklistedrob-prog/KDP-E-Book-Generatorcrazyrob425/KDP-E-Book-Generator(or whatever name you pick), send me the fork repo name.Then I can work against the fork and you’ll get:
src-tauri/recreatedWhat I need from you
Reply with either:
crazyrob425/KDP-E-Book-Generator” (best), orOnce you give me the fork repo path, I’ll proceed with the full rebuild there and keep it as close as possible to your current project structure and vibe.
User: i forked it: crazyrob425/KDP-E-Book-Generator
Assistant: I’ve started the Copilot coding agent work on your fork
crazyrob425/KDP-E-Book-Generatorto add the full Tauri v2 wrapper, Rust-only SQLite backend, job progress events, and a Windows MSI GitHub Actions build workflow; you can track progress at the task link below.User: @copilot Accepted Confirmation: Are you sure?
Assistant: I’m proceeding on your fork and kicking off the work to add the Tauri v2 wrapper + Rust SQLite backend + Windows MSI GitHub Actions build. You can track it here:
While that runs, if you want the result to feel exactly like your current app on Windows, tell me two preferences (one line is fine):
remember size/position(yes/no)%AppData%(default) or a folder next to the project filesUser: yes, in appdata
Assistant: Waiting on your response in the confirmation dialog to continue updating the in-progress PR with Windows window state persistence and AppData-based storage.
User: @copilot Accepted Confirmation: Are you sure?
Assistant: Got it: persist window size/position = yes, and store everything under AppData.
I can’t apply more repo changes from this message (I’m temporarily unable to run the repo-update automation right now), but here’s exactly what will be done in the PR so you know it’s handled correctly:
%AppData%\\Binary Books E-Crafter\\...or the Tauri app data path), not next to your project files.If you want me to match your preference precisely, confirm one detail:
yes/noUser: yes
Assistant: Window-state persistence (with off-screen clamping) + AppData-only storage is now queued to be implemented on `crazyrob425/KDP-E-Book-G...
This pull request was created from Copilot chat.
🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.