Skip to content

Conversation

timon-schelling
Copy link
Collaborator

@timon-schelling timon-schelling commented Aug 9, 2025

  • Improve save document semantics
    • Removes name from values saved in document (No migration should be needed)
    • Instead uses file name (without .graphite) as the document name
    • Document name is updated when the document path is updated
    • Document path is reset if the name is change by the user
  • Add Save as action
    • Adds Save As Menu entry and shortcut
    • Issue: Save as shourtcut Ctrl + Shift + S not working (help wanted)

@TrueDoctor
Copy link
Member

@Keavon We fixed the open issue this is now waiting on your thoughts regarding how the save process flow should work and if the implemented behaviour is desired

@GraphiteEditor GraphiteEditor deleted a comment from github-actions bot Aug 11, 2025
@Keavon
Copy link
Member

Keavon commented Aug 11, 2025

Let's discuss in a call so I know more about what's to be considered.

Also, this PR has a conflict.

This comment was marked as off-topic.

@TrueDoctor
Copy link
Member

@Keavon what is the status on this? I can review the code you just have to say if you are happy with the behaviour

@Keavon Keavon changed the title Improve save document semantics and add save as action Improve document save semantics and add "Save As…" action Aug 20, 2025
@Keavon Keavon changed the title Improve document save semantics and add "Save As…" action Desktop: Add the File > Save As… on desktop Aug 20, 2025
@TrueDoctor TrueDoctor changed the title Desktop: Add the File > Save As… on desktop Desktop: Add File > Save As… Aug 20, 2025
@timon-schelling timon-schelling enabled auto-merge (squash) August 20, 2025 09:59
@timon-schelling timon-schelling merged commit e70862b into master Aug 20, 2025
4 checks passed
@timon-schelling timon-schelling deleted the improve-save-document branch August 20, 2025 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants