celluloid icon indicating copy to clipboard operation
celluloid copied to clipboard

FR: implement portals

Open secretmango opened this issue 1 year ago • 7 comments

Celluloid is literally the only Wayland Video Player and thats great! thank you!

would be cool to have file-Portal support too, to have it fully secure and modern

secretmango avatar Dec 30 '23 19:12 secretmango

We're already using GtkFileChooserNative, which will automatically use file portals if they're available.

gnome-mpv avatar Jan 01 '24 03:01 gnome-mpv

Hm weird, on KDE I am not sure if it is a Portal. Do Desktops only have their native Portal or also another one possible?

I am pretty sure it only has the KDE portal.

secretmango avatar Jan 03 '24 13:01 secretmango

You may have to install the portal separately if you haven't already done so. On some (most?) distros, installing the desktop doesn't automatically install the portal.

Do Desktops only have their native Portal or also another one possible?

From my understanding, it should be possible to install portals other than the one that's made for your desktop, although they may not integrate as well.

gnome-mpv avatar Jan 09 '24 16:01 gnome-mpv

So then its an issue with the App. I am on Fedora Kinoite, did not install some KDE stuff on a barebones Distro.

It integrates the KDE xdg-desktop-portal and not the GNOME one, and normally for example in Firefox that works

secretmango avatar Jan 10 '24 19:01 secretmango

@gnome-mpv As a suggestion I think You should move to the new GtkFileDialog, there are many known bugs that GtkFileChooserNative doesn't work with portals properly. The snap package also faces the same issue. I requested the packager to enforce GDK_DEBUG=portals, but he denied it pointing out that it doesn't work on old portals from Ubuntu 22.04. I think, this migration might help to fix these portal related issues. There are several posts about it

https://forum.snapcraft.io/t/doesnt-use-the-xdg-desktop-portal/34360

https://forum.snapcraft.io/t/request-to-autoconnect-removable-media-for-gnome-text-editor/37485

The same problem was with gnome-text-editor, got fixed by GtkFileDialog.

soumyaDghosh avatar Jan 20 '24 08:01 soumyaDghosh

@soumyaDghosh I have been working on GtkFileDialog migration but ran into problems with dark themes. For some reason, GtkFileDialog doesn't respect per-application dark theme settings, although it respects global dark theme just fine. This is problematic for us because we use dark theme by default; a lot of users would end up seeing light-themed file dialog even though the main window is dark-themed, which doesn't look great.

gnome-mpv avatar Jan 20 '24 18:01 gnome-mpv

Thanks for your input @gnome-mpv, hope it can be fixed asap...but, the UI theme issue can be undermined. There is this video-player clapper, it also has this same kind of situation.

soumyaDghosh avatar Jan 20 '24 19:01 soumyaDghosh