Bug 420744

Summary:	Taking a rectangular region screenshot with Spectacle of desktop grid, makes desktops become grids
Product:	[Plasma] kwin	Reporter:	postix <postix>
Component:	effects-desktop-grid	Assignee:	KWin default assignee <kwin-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	plasma-bugs, postix
Priority:	NOR	Flags:	postix: X11+
Version:	5.22.1
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/kwin/commit/7a4cabf3287e82e7d1d6ba84b8b059ab470f9f42	Version Fixed In:	5.25
Sentry Crash Report:
Attachments:	Screen recording.

Description postix 2020-04-29 09:56:51 UTC

SUMMARY

It is not possible to take a screenshot of the grid view, when Spectacle in rectangular mode. 
Instead something weird happens.


STEPS TO REPRODUCE
1. Open spectacle, set area to "rectangular region" and delay to 5 seconds, so you have enough time to open the grid view
2. Press "Take new screenshot"

OBSERVED RESULT
A screenshot of the grid view is taken as is, allowing to select a region.

EXPECTED RESULT
Each desktop displayed becomes the before seen grid itself. Clicking somewhere goes back to the usual grid view.


SOFTWARE/OS VERSIONS
Operating System: Manjaro Linux 
KDE Plasma Version: 5.18.4
KDE Frameworks Version: 5.70.0
Qt Version: 5.14.2
Kernel Version: 5.6.7-1-MANJARO


ADDITIONAL INFORMATION
Scaling: 1.5

Comment 1 postix 2020-04-29 10:08:50 UTC

Created attachment 127984 [details]
Screen recording.

Edit: It's actually possible to take a screenshot of a rectangular region after clicking on one of the virtual desktops, which display a grid.

Please see the video demonstration.

Comment 2 postix 2021-06-16 18:11:46 UTC

At least on X11 I can still reproduce it.

Comment 3 Marco Martin 2022-05-06 10:47:57 UTC

Git commit 7a4cabf3287e82e7d1d6ba84b8b059ab470f9f42 by Marco Martin.
Committed on 06/05/2022 at 10:44.
Pushed by mart into branch 'master'.

QML version of the Desktop Grid effect

Replace completely the old desktop grid effect with a QML version.
Aims to feature parity and be a change as transparent as possible for the user.
Related: bug 433071, bug 452625, bug 443971, bug 437121, bug 452925, bug 437928, bug 452439, bug 288530, bug 450254, bug 450106, bug 447832, bug 449960, bug 416576, bug 441862, bug 444859, bug 445999, bug 422117, bug 404627, bug 435483, bug 435482, bug 427055, bug 333445, bug 429120, bug 427391, bug 409295, bug 294322, bug 356955
FIXED-IN: 5.25

M  +5    -0    src/effects.cpp
M  +10   -5    src/effects/desktopgrid/CMakeLists.txt
D  +0    -1571 src/effects/desktopgrid/desktopgrid.cpp
D  +0    -186  src/effects/desktopgrid/desktopgrid.h
D  +0    -32   src/effects/desktopgrid/desktopgrid.kcfg
M  +6    -14   src/effects/desktopgrid/desktopgrid_config.cpp
M  +2    -2    src/effects/desktopgrid/desktopgrid_config.h
M  +68   -144  src/effects/desktopgrid/desktopgrid_config.ui
A  +32   -0    src/effects/desktopgrid/desktopgridconfig.kcfg
M  +5    -1    src/effects/desktopgrid/desktopgridconfig.kcfgc
A  +342  -0    src/effects/desktopgrid/desktopgrideffect.cpp     [License: GPL(v2.0+)]
A  +108  -0    src/effects/desktopgrid/desktopgrideffect.h     [License: GPL(v2.0+)]
M  +5    -4    src/effects/desktopgrid/main.cpp
D  +0    -26   src/effects/desktopgrid/main.qml
M  +1    -0    src/effects/desktopgrid/metadata.json
A  +255  -0    src/effects/desktopgrid/qml/DesktopView.qml     [License: GPL(v2.0+)]
A  +193  -0    src/effects/desktopgrid/qml/main.qml     [License: GPL(v2.0+)]
M  +22   -5    src/effects/private/qml/WindowHeap.qml
M  +21   -3    src/libkwineffects/kwineffects.h
M  +4    -1    src/libkwineffects/kwinquickeffect.cpp

https://invent.kde.org/plasma/kwin/commit/7a4cabf3287e82e7d1d6ba84b8b059ab470f9f42