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

Update NFDe implementation to v1.1.1 #96

Merged
merged 1 commit into from
Feb 19, 2024

Conversation

TomChapple
Copy link
Contributor

@TomChapple TomChapple commented Feb 18, 2024

In preparation for distributing Gearsystem as a Flatpak application in #95, this pull request upgrades the source files for NFDe to the v1.1.1 release. Most notably, this also includes an implementation in nfd_portal.cpp that supports the use of the Portal API used by Flatpak applications. This is necessary to have native file chooser dialogs that grant permission without needing machine-wide filesystem access.

As I run this program on a Linux machine, I am unsure if the build works on the other platforms. Any verification that it continues to work as intended on those platforms would be greatly appreciated.

This updates the existing NFD implementation to [v1.1.1][1], including
additional implementations for the DBus Portal API, which is useful for
sandboxed applications to be able to choose files.

As the interface into this library is the same as it was before, no
existing appliction code needs to be updated.

[1]: https://github.com/btzy/nativefiledialog-extended/releases/tag/v1.1.1
@drhelius
Copy link
Owner

You can check the GH Action build for each platform. It's building fine.

But I'm going to quickly test it to check if there is any problem on runtime.

I'll get back to you.

@drhelius
Copy link
Owner

It looks like it's working fine.

@drhelius drhelius merged commit aa79432 into drhelius:master Feb 19, 2024
7 checks passed
@TomChapple TomChapple deleted the nfde-1.1.1 branch February 19, 2024 10:18
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.

2 participants