Skip to content

Fix file saving for FreeDesktop Portals #3089

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.

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

Merged
merged 2 commits into from
Feb 8, 2024
Merged

Conversation

amolenaar
Copy link
Member

PR Type

What kind of change does this PR introduce?

  • Bug fix
  • Feature
  • Chore (refactoring, formatting, local variables, other cleanup)
  • Documentation content changes

What is the current behavior?

Files are not properly saved from Flatpaks if you do not define the .gaphor extension to the file.

Issue Number: #3085

What is the new behavior?

Instead of adding an extension (which does not sync via XDP portals), provide a default name with extension.

This allows users to easily change the file name, with the extension already visible in the file entry field.

Do no longer "magically" add an extension.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@amolenaar amolenaar requested a review from danyeaw February 7, 2024 22:19
@github-actions github-actions bot added the python Pull requests that update Python code label Feb 7, 2024
Instead of adding an extension (which does not sync via
XDP portals), provide a default name with extension.

This allows users to easily change the file name, with the extension
already visible in the file entry field.

Do no longer "magically" add an extension.
@amolenaar amolenaar marked this pull request as draft February 7, 2024 22:22
Explicit extension is no longer needed, and it requires a file name now.
@amolenaar amolenaar force-pushed the fix-flatpak-file-dialog branch from f9e1f3c to 705854a Compare February 7, 2024 22:42
@amolenaar amolenaar marked this pull request as ready for review February 7, 2024 22:43
Copy link
Member

@danyeaw danyeaw left a comment

Choose a reason for hiding this comment

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

This looks great, thanks @amolenaar!

@danyeaw danyeaw added fix A fix for a bug and removed python Pull requests that update Python code labels Feb 8, 2024
@danyeaw danyeaw merged commit b6d7dfc into main Feb 8, 2024
@danyeaw danyeaw deleted the fix-flatpak-file-dialog branch February 8, 2024 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix A fix for a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants