Skip to content
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

Use an absolute filename for the file save dialog on Linux #7108

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pelya
Copy link
Contributor

@pelya pelya commented Dec 5, 2024

First time contributor checklist:

Contributor checklist:

  • My contribution is not related to translations.
  • My commits are in nice logical chunks with good commit messages
  • My changes are rebased on the latest main branch
  • A npm run ready run passes successfully (more about tests here)
  • My changes are ready to be shipped to users

Description

This fixes #6912.
On Gnome the save dialog works with any filename, on KDE Plasma the save dialog opens with an empty filename input field, unless the application uses an absolute file path instead of only the filename. This commit will use the ~/Downloads directory as the default path, which seems like a better choice then dumping files into the user's home dir.

Copy link

@palant palant left a comment

Choose a reason for hiding this comment

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

I’m not a Signal developer but if you want this change merged you should probably get it into a decent shape.

app/main.ts Outdated Show resolved Hide resolved
app/main.ts Show resolved Hide resolved
app/main.ts Show resolved Hide resolved
app/main.ts Outdated Show resolved Hide resolved
This fixes signalapp#6912.
On Gnome the save dialog works with any filename, on KDE Plasma
the save dialog opens with an empty filename input field,
unless the application uses an absolute file path instead of only the filename.
The default save location is the Downloads directory in the user's home directory.
@pelya pelya force-pushed the kde-save-dialog-fix branch from 601ac41 to ad3e940 Compare December 13, 2024 10:52
@yash-signal yash-signal self-requested a review December 13, 2024 20:35
@yash-signal yash-signal self-assigned this Dec 13, 2024
@arlen-kleinsasser
Copy link

As a user who is experiencing bug 6912. wanted to give this a bump. Is there any reason this fix hasn't been merged? And thank you @pelya for writing this fix.

@pelya
Copy link
Contributor Author

pelya commented Jan 27, 2025

Hey, this fix is still not merged, and today I have discovered again that I cannot save a file from Signal without clicking through my whole filesystem and typing some bogus filename. Could you please merge it, let's say, before year 2025 ends?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Unable to save media or files via Desktop on Linux, while Android works fine
4 participants