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

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

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

Conversation

ulefebvr
Copy link

Make it compatible with the flatpak requirements

Make it compatible with the flatpak requirements
@davidmhewitt
Copy link
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
Copy link
Author

ulefebvr commented Sep 1, 2021

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

@sarojbelbase
Copy link

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

@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

@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