[PR #602] [MERGED] Fix for window size save overwriting config on close #615

New issue

Closed

opened 2026-02-28 14:33:38 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 14:33:38 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jpochyla/psst/pull/602
Author: @jacksongoode
Created: 4/20/2025
Status: ✅ Merged
Merged: 4/20/2025
Merged by: @jacksongoode

Base: main ← Head: config-save-fox

📝 Commits (1)

6fc714a Fix

📊 Changes

2 files changed (+0 additions, -3 deletions)

View changed files

📝 psst-gui/src/delegate.rs (+0 -2)
📝 psst-gui/src/main.rs (+0 -1)

📄 Description

There was a bug here with the way the config was being written. We don't need to write any config unless we close down nicely and we will save the window size if we can. Otherwise don't do it.

Resolves https://github.com/jpochyla/psst/issues/598

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jpochyla/psst/pull/602 **Author:** [@jacksongoode](https://github.com/jacksongoode) **Created:** 4/20/2025 **Status:** ✅ Merged **Merged:** 4/20/2025 **Merged by:** [@jacksongoode](https://github.com/jacksongoode) **Base:** `main` ← **Head:** `config-save-fox` --- ### 📝 Commits (1) - [`6fc714a`](https://github.com/jpochyla/psst/commit/6fc714a6394b14ce31612eb8c7288151fdece648) Fix ### 📊 Changes **2 files changed** (+0 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `psst-gui/src/delegate.rs` (+0 -2) 📝 `psst-gui/src/main.rs` (+0 -1) </details> ### 📄 Description There was a bug here with the way the config was being written. We don't need to write any config unless we close down nicely and we will save the window size if we can. Otherwise don't do it. Resolves https://github.com/jpochyla/psst/issues/598 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>