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

Add gnome-based wayland universal screenshot adapter #3215

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jack9603301
Copy link
Contributor

@jack9603301 jack9603301 commented Jun 2, 2023

By default, flameshot uses the program's default dbus protocol interface to communicate with gnome's screenshot protocol, but this is now broken on some gnome installations, so we'll use gnome-screenshot as gnome's wayland screenshot adapter and then communicate directly with the gnome screenshot component!

Designed to address: #3213

By default, flameshot uses the program's default dbus protocol interface
to communicate with gnome's screenshot protocol, but this is now broken
on some gnome installations, so we'll use gnome-screenshot as gnome's
wayland screenshot adapter and then communicate directly with the gnome
screenshot component!

To enable the GNOME-based generic wayland screenshot adapter, enable it
using the following cmake parameter:

```
-DUSE_WAYLAND_GNOME=true
```
@jack9603301
Copy link
Contributor Author

@mmahmoudian This solution is pending review, and it seems to work fine in GNOME

@tppawelgrzesik
Copy link

tppawelgrzesik commented Feb 23, 2024

@jack9603301 thanks for the fix!

I have tested this on my setup and after installing gnome-screenshot (is this a legacy package? It looks like an old-version of the already build-in screenshot tool) flameshot finally works again.

I'm on Ubuntu 23.10 6.5.0-21 wayland.

However I noticed a performance issue, I'm running two 4k screens and when selecting the area to screenshot it lags terribly. Feels like 2 FPS. But at least it works 👍

@xarpen
Copy link

xarpen commented Mar 29, 2024

Great work, this finally makes flameshot work under arch/gnome/wayland!
Would be nice if this can be merged into mainstream.

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.

3 participants