-
Notifications
You must be signed in to change notification settings - Fork 9
integration: describe how to fall back on non-xdg-portal file chooser #61
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
Conversation
src/integration.md
Outdated
Chooser: | ||
|
||
1. Set the environment variable `GDK_DEBUG=no-portals` | ||
2. Create a `/usr/share/xdg-desktop-portal/labwc-portals.conf` with the line `org.freedesktop.impl.portal.FileChooser=none` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May be better to advise to create a configuration file in the current user home (https://flatpak.github.io/xdg-desktop-portal/docs/portals.conf.html) based on https://github.com/labwc/labwc/blob/master/data/labwc-portals.conf with the mentioned addition.
Chances are there that a system configuration file already exists, at least that is how Labwc is packaged for Fedora.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay. Good suggestions. Only read this after below fixup. Will carry on tomorrow.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, since it is a GTK issue, and gtk apps are in all the wayland compositors, maybe portals.conf
would be preffered instead of labwc-portals.conf
. I agree about the file location, preferring ~/.config/xdg-desktop-portal/
($XDG_CONFIG_HOME/xdg-desktop-portal/
, as mentioned in the link shared).
Not sure if the issue happens in other compositors, its just a suggestion.
3d7e8c5
to
73aca48
Compare
Thanks. How about this then? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Small suggestion.
src/integration.md
Outdated
xdg-portal technology which may not work depending on your system setup. There | ||
are at least two ways to fallback to a 'normal' File Chooser: | ||
|
||
1. Set the environment variable `GDK_DEBUG=no-portals` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion: ..
in e.g. `~/.config/labwc/environment`
73aca48
to
8954d41
Compare
Related to: labwc/labwc#2300
Cc @jp7677