Skip to content

fix: save drafts on quit#1424

Merged
mavonx merged 2 commits into
floatpane:masterfrom
andrinoff:fix/save-on-quit
Jun 4, 2026
Merged

fix: save drafts on quit#1424
mavonx merged 2 commits into
floatpane:masterfrom
andrinoff:fix/save-on-quit

Conversation

@andrinoff
Copy link
Copy Markdown
Member

What?

Checks if the composer has contents at the moment of interrupt signal (ctrl+c). If so, it saves the draft before quitting

Why?

It is easy to accidentally quit, and it was unforgiving, without saving anything. Especially including that the default quit keybind is the same as "copy" on linux and windows (ctrl+c)

Signed-off-by: drew <me@andrinoff.com>
@andrinoff andrinoff requested a review from a team as a code owner June 3, 2026 12:55
@floatpanebot floatpanebot added bug Something isn't working os/linux Linux-specific os/windows Windows-specific area/tui Terminal UI / view layer size/S Diff: 11–50 lines backport/v1 Merge to master and backport to release/v1 labels Jun 3, 2026
@andrinoff andrinoff requested a review from mavonx June 4, 2026 07:06
@mavonx mavonx added this pull request to the merge queue Jun 4, 2026
Merged via the queue into floatpane:master with commit 0527f4c Jun 4, 2026
29 checks passed
@floatpanebot floatpanebot added the backported Cherry-picked onto release/v1 label Jun 4, 2026
floatpanebot pushed a commit that referenced this pull request Jun 4, 2026
## What?

Checks if the composer has contents at the moment of interrupt signal
(`ctrl+c`). If so, it saves the draft before quitting

## Why?

It is easy to accidentally quit, and it was unforgiving, without saving
anything. Especially including that the default quit keybind is the same
as "copy" on linux and windows (`ctrl+c`)

Signed-off-by: drew <me@andrinoff.com>
(cherry picked from commit 0527f4c)
@floatpane floatpane deleted a comment from andrinoff Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/tui Terminal UI / view layer backport/v1 Merge to master and backport to release/v1 backported Cherry-picked onto release/v1 bug Something isn't working os/linux Linux-specific os/windows Windows-specific size/S Diff: 11–50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants