Bug 443233

Summary:	GHNS dialog being modal can block all of Plasmashell
Product:	[Frameworks and Libraries] frameworks-knewstuff	Reporter:	Gabriel <gabriel>
Component:	general	Assignee:	Dan Leinir Turthra Jensen <admin>
Status:	RESOLVED FIXED
Severity:	normal	CC:	kdelibs-bugs, nate
Priority:	NOR	Keywords:	usability
Version:	5.86.0
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/frameworks/knewstuff/-/commit/ea19fa6e824650f3257e8047d6f90e01899b2e03	Version Fixed In:	5.93
Sentry Crash Report:

Description Gabriel 2021-10-02 20:21:44 UTC

SUMMARY
Opening the wallpaper GHNS window locks everything on the desktop until it's closed again

STEPS TO REPRODUCE
1. Open "Configure Desktop and Wallpaper..."
2. Click "Get New Wallpapers..."
3. Try to do anything on the desktop, e.g. open context menu, desktop icons, etc.

OBSERVED RESULT
You are unable to do anything with the desktop until GHNS is closed again

EXPECTED RESULT
You can still interact with the desktop

SOFTWARE: 
OS: Neon Testing Edition
KDE Plasma Version: 5.22.90
KDE Frameworks Version: 5.87.0
Qt Version: 5.15.3

ADDITIONAL INFORMATION
It affects the most when it is open on another virtual desktop, since you will not find the blocking window in the taskbar. The same happens with the wallpaper plugins GHNS window.

Comment 1 Nate Graham 2021-10-04 22:36:30 UTC

This happens because it's marked as a modal dialog, and modal dialogs block their parent app until closed. When the parent app is Plasma, Plasma gets blocked while it's open.

Options:
- Make it not be a modal dialog
- Make the modality of the dialog configurable and have Plasma open it in a non-modal way

Comment 2 Gabriel 2022-03-15 19:11:32 UTC

This bug appears to be fixed now (Plasma 5.24.3, Frameworks 5.91). Since GHNS won't even be a dialog, but a window, from Frameworks 5.93 and onward, I feel confident that this bug is past us for good – even if I can't pin-point exactly when it got fixed. Thanks devs!

Comment 3 Nate Graham 2022-03-15 19:14:37 UTC

You are correct about all of that! The exact commit was https://invent.kde.org/frameworks/knewstuff/-/commit/ea19fa6e824650f3257e8047d6f90e01899b2e03, made by Alexander Lohnau.