Bug 465784 - "No delay" setting actually includes a built-in 200ms delay
Summary: "No delay" setting actually includes a built-in 200ms delay
Status: RESOLVED FIXED
Alias: None
Product: Spectacle
Classification: Applications
Component: General (show other bugs)
Version: 21.12.3
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Boudhayan Gupta
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2023-02-15 18:32 UTC by Hubert Kaschulke
Modified: 2023-04-14 21:44 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 23.04
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Hubert Kaschulke 2023-02-15 18:32:27 UTC
SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1. Setting: Delay: NO DELAY
2. 
3. 

OBSERVED RESULT
DELAYED of ca. 2 seconds


EXPECTED RESULT
NO DELAY  (should start IMMEDIATELY)

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma:  openSUSE Leap 15.4
(available in About System)
KDE Plasma Version: 5.24.4
KDE Frameworks Version: Version 5.90.0
Qt Version: Version 5.15.2 (built against 5.15.2)

ADDITIONAL INFORMATION
Comment 1 guimarcalsilva 2023-02-16 02:09:14 UTC
I took my phone, put a real-time clock on the screen (https://clock.zone), and recorded a slow-mo video from the screen. Indeed, comparing the time the window disappeared to the time that appeared on the screenshot there was a delay of ~0.1 seconds. 

I would say that is imperceptible in normal usage, but it's indeed there. I will not change the status to CONFIRMED because I don't know if this is a real bug or just a matter of the processing time it takes to save the screenshot. A comment from a developer would be good.

I'm on Neon Unstable (5.27.80) with Frameworks 104.
Comment 2 Nate Graham 2023-02-17 20:21:14 UTC
There is always a 200ms delay even when using "No Delay", yeah. I believe Noah is looking into it.
Comment 3 Bug Janitor Service 2023-04-11 16:41:55 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/3961
Comment 4 Noah Davis 2023-04-11 19:23:04 UTC
Git commit 616783d892db0410497356409698a81311610fc6 by Noah Davis.
Committed on 11/04/2023 at 19:11.
Pushed by ndavis into branch 'master'.

Blacklist Spectacle for all window open/close effects

Spectacle needs to be blacklisted in order to stay out of its own
screenshots.
Related: bug 467890, bug 463105

M  +5    -1    src/effects/fade/package/contents/code/main.js
M  +3    -0    src/effects/glide/glide.cpp
M  +5    -1    src/effects/scale/package/contents/code/main.js

https://invent.kde.org/plasma/kwin/commit/616783d892db0410497356409698a81311610fc6
Comment 5 Noah Davis 2023-04-11 19:23:58 UTC
Git commit b090a7cc9e3b2a70e88c47b5fb6add8599bc7c0d by Noah Davis.
Committed on 11/04/2023 at 19:23.
Pushed by ndavis into branch 'Plasma/5.27'.

Blacklist Spectacle for all window open/close effects

Spectacle needs to be blacklisted in order to stay out of its own
screenshots.
Related: bug 467890, bug 463105


(cherry picked from commit 616783d892db0410497356409698a81311610fc6)

M  +5    -1    src/effects/fade/package/contents/code/main.js
M  +3    -0    src/effects/glide/glide.cpp
M  +5    -1    src/effects/scale/package/contents/code/main.js

https://invent.kde.org/plasma/kwin/commit/b090a7cc9e3b2a70e88c47b5fb6add8599bc7c0d
Comment 6 Noah Davis 2023-04-11 22:20:06 UTC
This will be fixed in version 23.04 if you have KWin 5.27.5 or greater.
Comment 7 Noah Davis 2023-04-14 21:44:09 UTC
Apparently KWin 5.27.4.1 has the required code in it, so I can reduce the minimum Plasma version to 5.27.4