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

Ksnip window always visible on screenshots on Gnome Wayland #375

Closed
g3blv opened this issue Jul 10, 2020 · 24 comments
Closed

Ksnip window always visible on screenshots on Gnome Wayland #375

g3blv opened this issue Jul 10, 2020 · 24 comments
Assignees
Labels
Projects

Comments

@g3blv
Copy link

g3blv commented Jul 10, 2020

It would be nice if the Ksnip app window was hidden while taking screenshots. At the moment I have to move the Ksnip app window to make sure it doesn't become part of the screenshot. Moving the window is harder on small screens since there is not a lot of room where it can be put to no be in the screenshot.

Not sure if this is a bug in the Appimage or a feature that could be added. I've tried looking for a setting to enable it but haven't found one.

Using
Appimage
Version: 1.7.0
Build: 1238-caa6fa9
OS: Fedora 32

@DamirPorobic
Copy link
Member

DamirPorobic commented Jul 10, 2020 via email

@g3blv
Copy link
Author

g3blv commented Jul 11, 2020

I made a short video. The issue seems more like the Ksnip app window is hidden for a short while and then comes back. I'm happy to provide more info or testing if needed.

Ksnip issue

@DamirPorobic
Copy link
Member

DamirPorobic commented Jul 11, 2020 via email

@g3blv
Copy link
Author

g3blv commented Jul 11, 2020

I wasn't able find "Freeze Background while snipping" but I found "Freeze image while snipping" and it is already disabled.

Enjoy your holiday ;)

ksnip_20200711-111054

@DamirPorobic
Copy link
Member

Oh, you're using Wayland maybe?

@g3blv
Copy link
Author

g3blv commented Jul 13, 2020

Yes I'm using Wayland.

@DamirPorobic DamirPorobic self-assigned this Jul 13, 2020
@DamirPorobic DamirPorobic added this to To do in v1.7.2 via automation Jul 13, 2020
@DamirPorobic
Copy link
Member

Ok, I can reproduce that but only under Gnome Wayland. Under Gnome X11 it seems to be working and under Plasma Wayland we can't take rect screenshots.

@DamirPorobic DamirPorobic moved this from To do to In progress in v1.7.2 Jul 13, 2020
@DamirPorobic
Copy link
Member

DamirPorobic commented Jul 13, 2020

I think I found the issue. Usually we minimize to the task bar and hide the window, but it looks like Wayland is not happy with minimizing. With the change below it's working for me in my Fedora VM.

image

Does Wayland/Gnome support minimizing application?

@g3blv
Copy link
Author

g3blv commented Jul 14, 2020

I tested Ksnip AppImage on Ubuntu 20.04 desktop using Wayland. Ubuntu is using minimize buttons on all windows. I was able to minimize Ksnip by clicking the minimize button on Ubuntu.

@DamirPorobic
Copy link
Member

Do you have there the same behavior that ksnip is visible on screenshots?

@DamirPorobic
Copy link
Member

Maybe we should generally switch setting visibility? I'll look into it tonight again, we should have a test version by end of the day.

@g3blv
Copy link
Author

g3blv commented Jul 14, 2020

Yes same behaviour.

@DamirPorobic
Copy link
Member

Can you test with latest appImage? For me it's working with this code change on fedora 31. Just give the CI a couple of minutes to finish building the binaries.

@g3blv
Copy link
Author

g3blv commented Jul 15, 2020

I had a look on https://github.com/ksnip/ksnip/releases/tag/continuous for the latest AppImage but there is only EXE, DEB, RPM, DMG and source code. I also looked in CI/Travis but couldn't find any AppImage there. Am I looking in the wrong places?

@DamirPorobic
Copy link
Member

Sorry, something went wrong with the upload script. I've restarted the build process and not it's there.

@g3blv
Copy link
Author

g3blv commented Jul 15, 2020

I tested the AppImage for 1.7.2 (it says build 12238-caa6fa9 in About - Version, I'm not sure if this updates for the continuous builds) now on Fedora 32. Still the same issue. Looks like the window is minimizing for a short while but then comes back again.

Ksnip issue 2

@DamirPorobic
Copy link
Member

Doesn't look right, the commit hash under about seems to be wrong. Sorry, will look into it after work.

@DamirPorobic
Copy link
Member

I get the latest version from the release page
image

@g3blv
Copy link
Author

g3blv commented Jul 16, 2020

Now I have the same version and commit and I can confirm that the Ksnip app minimize when I take a screenshot. Great! Thanks! I really enjoy using Ksnip. I used to use Shutter but it doesn't support Wayland. Ksnip works just as well for me on Wayland. Should I close this issue?

@DamirPorobic
Copy link
Member

I want to do a a small refactoring as I have changed the behavior for all version but the fix we need actually only for gnome Wayland. After I have pushed that changed I will close it. I might ask you to test the final fix once again. Thanks for confirming.

Yes, I hear a lot of people coming from Shutter and I kind of target those users with features and try to implement what they had with Shutter in order to make the transition easier.

@g3blv
Copy link
Author

g3blv commented Jul 16, 2020

Sure I'm happy to test after refactoring.

Great, my transition have been very smooth from Shutter to Ksnip.

@DamirPorobic DamirPorobic changed the title Hide Ksnip app window when taking screenshots Ksnip window always visible on screenshots on Gnome Wayland Jul 19, 2020
@DamirPorobic
Copy link
Member

I have pushed a new commit, the new behavior should only be used for gnome wayland and all other platforms end distros should be using the old behavior. You could just double check that it's still working as expected.

v1.7.2 automation moved this from In progress to Done Jul 19, 2020
@g3blv
Copy link
Author

g3blv commented Jul 19, 2020

I've tested latest continues build

Version: 1.7.2-continuous
Build: 1287-070cab6

On Fedora 32 using Gnome and Wayland

And it is working as expected.

Thanks for fixing the issue.

@DamirPorobic
Copy link
Member

Thanks for testing and confirming :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
v1.7.2
  
Done
Development

No branches or pull requests

2 participants