Bug 465784

Summary:	"No delay" setting actually includes a built-in 200ms delay
Product:	[Applications] Spectacle	Reporter:	Hubert Kaschulke <dgplfz5sri-Bu99AXCtTQ>
Component:	General	Assignee:	Boudhayan Gupta <me>
Status:	RESOLVED FIXED
Severity:	normal	CC:	kde, nate, noahadvs
Priority:	NOR	Keywords:	usability
Version:	21.12.3
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	3be1c1d734786bc06ea2bbfb9088fe29f0f024c6	Version Fixed In:	23.04
Sentry Crash Report:

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