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

Move away from gnome-shell screenshot API #185

Closed
ciciolina opened this issue Aug 11, 2020 · 9 comments
Closed

Move away from gnome-shell screenshot API #185

ciciolina opened this issue Aug 11, 2020 · 9 comments
Assignees

Comments

@ciciolina
Copy link

ciciolina commented Aug 11, 2020

Currently, the only supported platform with the QR scanner is GNOME, can you add a setting option or embedded to support more screenshot app's? like flameshot, Shutter or other alternatives. This will allow us to get more support and cover more DE for all users that have others DE different or non compatible with gnome.

This is the error we get on a non-gnome desktop environment:

GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: the name org.gnome.Shell.Screenshot was not provided by any .service files

Pd. i have gnome gnome-screenshot working in my system but i dont have gnome-shell, the app call this shell to let me take a QR screenshot, its impossible to me (and others users), install this shell without killing the DE.

System info:
OS: Arch Linux x86_64
Kernel: 5.7.12-arch1-1
DE: Cinnamon

Your app its amazing!
Thanks!

@paolostivanin
Copy link
Owner

Hi,
I'll look into it, sure! It will take me some time though, because I have to install VMs, DEs, etc 😄

@ciciolina
Copy link
Author

Thanks i wait for future updates, many thanks for your time into this... if you have any instruction or something i can try only say me, kisses!

Pd. I change my DM to XFCE and send same error... on Gnome works perfectly.

@paolostivanin paolostivanin added this to To Do in Release 3.0.0 Oct 19, 2020
@MrMikeJJ
Copy link

Ps. I change my DM to XFCE and send same error... on Gnome works perfectly.

I Googled this yesterday. it looks like you mentioned the reason in your first post. Gnome Shell is the dependacy which needs to be installing or to be running (not sure which).

@paolostivanin
Copy link
Owner

The current implementation requires gnome-shell to be running because it uses its dbus api.
With v3 I'll add support for another other 3rd party tool (not sure which one ATM)

@paolostivanin paolostivanin added this to the 3.0.0 milestone Feb 25, 2021
@paolostivanin paolostivanin changed the title QR issue - please add more screenshot support instead of GNOME shell's one Move away from gnome-shell screenshot API Nov 26, 2021
@paolostivanin
Copy link
Owner

This is now more important than ever because starting from GNOME 41 this dbus interface will no longer be accessible by third party apps.

@paolostivanin paolostivanin modified the milestones: 3.0.0, 2.5.0 Nov 30, 2021
@paolostivanin paolostivanin removed this from To Do in Release 3.0.0 Nov 30, 2021
@paolostivanin paolostivanin added this to To do in Release 2.5.0 Dec 16, 2021
@paolostivanin paolostivanin moved this from To do to In progress in Release 2.5.0 Feb 19, 2022
@paolostivanin
Copy link
Owner

falmeshot removed most of its dbus api, so it's no longer an option. Need to find a valid and stable alternative.

@paolostivanin
Copy link
Owner

It seems that there's no shared standard to take screenshots across DE. People are working on some kind of screenshot portal, but it's not here yet.
I guess I will simply remove this feature from OTPClient, since there is no real alternative ATM.

@paolostivanin
Copy link
Owner

The screenshot feature will be removed until a stable and cross DE solution will come out.

paolostivanin added a commit that referenced this issue Feb 19, 2022
see #185 for more info
paolostivanin added a commit that referenced this issue Mar 30, 2022
see #185 for more info
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants