Skip to content

[FLATPAK] Meet the requirement of OS6 #73#75

Open
ulefebvr wants to merge 1 commit into
davidmhewitt:masterfrom
ulefebvr:patch-1
Open

[FLATPAK] Meet the requirement of OS6 #73#75
ulefebvr wants to merge 1 commit into
davidmhewitt:masterfrom
ulefebvr:patch-1

Conversation

@ulefebvr

Copy link
Copy Markdown

Make it compatible with the flatpak requirements

Make it compatible with the flatpak requirements
@davidmhewitt

Copy link
Copy Markdown
Owner

Thanks!

However, this requires some more thought.

The reason I haven't ported this to Flatpak myself yet is that this won't fully work. The way Clipped sets up a custom global keyboard shortcut won't work from inside a sandbox and the ability to open Clipped with a keyboard shortcut is, in my opinion, one of the most useful features.

@ulefebvr

ulefebvr commented Sep 1, 2021

Copy link
Copy Markdown
Author

Indeed I had to create a system side keyboard shortcut to make it work.
I will dig into it.

@sarojbelbase

Copy link
Copy Markdown

The reason I haven't ported this to Flatpak myself yet is that this won't fully work. The way Clipped sets up a custom global keyboard shortcut won't work from inside a sandbox and the ability to open Clipped with a keyboard shortcut is, in my opinion, one of the most useful features.

Actually, let the user manually map their own keyboard shortcut; It's a neat feature but this can go manual. I absolutely love this project because of its simplicity. Please continue working on it. It just works.

@Korben11

Copy link
Copy Markdown

@davidmhewitt I see that the Issue with the global shortcut and Flatpack seems to be resolved:
Issue: flatpak/xdg-desktop-portal#624
Resolved by: flatpak/xdg-desktop-portal#711

@eddy-geek

Copy link
Copy Markdown

@Korben11 to be clear, the portal seems only implemented KDE so far, but it is indeed the future™ (and probably better than nothing for flatpak).

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.

5 participants