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

Set StartupNotify to false #218

Merged
merged 1 commit into from Sep 17, 2019

Conversation

@Shatur95
Copy link
Contributor

commented Sep 16, 2019

Add StartupNotify=false to the .desktop file like in Spectacle. On KDE Plasma without this I can see jumping icon that appear on screenshots.
изображение

@DamirPorobic

This comment has been minimized.

Copy link
Owner

commented Sep 17, 2019

Thanks for opening this PR. Out of curiosity, where does this behavior happen? I'm using also KDE Plasma and haven't noticed it yet.

@Shatur95

This comment has been minimized.

Copy link
Contributor Author

commented Sep 17, 2019

This happens all the time when I launch the application (It doesn’t matter if I launch from a hotkey or from a menu).
Maybe you disabled launch feedback globally? Setting StartupNotify=false just disables it for one app.

@DamirPorobic

This comment has been minimized.

Copy link
Owner

commented Sep 17, 2019

Strange, I thought this is intended behavior. With ksnip I don't see it on my notebook, it opens too quick, but with larger applications, like, CLion I do see a jumping icon next to the mouse cursor. Is there a reason why you want to disable it?

@Shatur95

This comment has been minimized.

Copy link
Contributor Author

commented Sep 17, 2019

Maybe you have faster hardware?
I sended this PR because I see jumping icon on every screenshot (when I using shortcuts for it). For example, Spectacle have StartupNotify=false by default too.

@DamirPorobic

This comment has been minimized.

Copy link
Owner

commented Sep 17, 2019

Now I get you, the icon is visible on your screenshots, that is indeed unfortunate. Just wanted to understand the issue you're trying to fix here.

@DamirPorobic DamirPorobic merged commit afd1f52 into DamirPorobic:master Sep 17, 2019
2 checks passed
2 checks passed
Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Shatur95

This comment has been minimized.

Copy link
Contributor Author

commented Sep 17, 2019

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.