Bug 454635

Summary: Hovering over panel applet closes edit mode
Product: [Plasma] plasmashell Reporter: Nicolas Fella <nicolas.fella>
Component: PanelAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: carlon.luca, nate, niccolo.venerandi, notmart, qydwhotmail
Priority: VHI Keywords: regression
Version: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.95
Sentry Crash Report:
Attachments: Screen recording
Correct screen recording

Description Nicolas Fella 2022-05-31 01:06:10 UTC
STEPS TO REPRODUCE
1. Right-click on a panel
2. Enter Edit mode
3. Hover over an applet in the panel

OBSERVED RESULT
The panel leaves edit mode

EXPECTED RESULT
Actions like removing the applet are shown

SOFTWARE/OS VERSIONS
KDE Plasma Version: master
KDE Frameworks Version: master
Qt Version: 5.15.4-kde

Works as expected on X11

ADDITIONAL INFORMATION
Comment 1 Nicolas Fella 2022-05-31 01:11:15 UTC
It works fine in 5.24.5
Comment 2 Nicolas Fella 2022-05-31 01:38:00 UTC
The issue seems to be with plasma-framework, in particular https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/500
Comment 3 Luca Carlon 2022-05-31 22:32:14 UTC
I have to say I'm not currently able to reproduce with my build. I'll try to rebuild the entire Plasma.
Comment 4 Nate Graham 2022-06-01 16:50:58 UTC
Hmm, cannot reproduce with today's git master.
Comment 5 Luca Carlon 2022-06-01 21:30:10 UTC
I rebuilt the entire Plasma from master and I still cannot reproduce. Could you provide a video of the issue? Maybe there is something different in our setup.
Comment 6 Nicolas Fella 2022-06-05 16:47:25 UTC
okay, removing the applet seems to work okay now. However, the UI for editing the panel itself (where you can move the panel, set height etc) still disappears when hovering over an applet in the panel. It didn't do that before
Comment 7 Nicolas Fella 2022-06-05 16:54:46 UTC
Created attachment 149484 [details]
Screen recording
Comment 8 Nicolas Fella 2022-06-05 16:55:41 UTC
Created attachment 149485 [details]
Correct screen recording
Comment 9 Nate Graham 2022-06-05 17:23:45 UTC
I have started being able to reproduce this. Just noticed it yesterday.
Comment 10 Luca Carlon 2022-06-05 20:08:04 UTC
Is this confirmed to be related to https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/500?
Comment 11 Nicolas Fella 2022-06-05 20:11:54 UTC
reverting/using the last commit before https://invent.kde.org/frameworks/plasma-framework/-/commit/affcdeed45755321d7d41c83375729abf9018d5f fixes it
Comment 12 Nate Graham 2022-06-05 20:53:25 UTC
I can confirm that.
Comment 13 Luca Carlon 2022-06-06 16:38:10 UTC
I tried on both X11 and Wayland but I'm not able to reproduce the issue in my build. But I'm sure I saw this some weeks ago.

If the problem appears when hovering, these two places come to my mind: https://invent.kde.org/frameworks/plasma-framework/-/blob/affcdeed45755321d7d41c83375729abf9018d5f/src/plasmaquick/dialog.cpp#L1483 and https://invent.kde.org/frameworks/plasma-framework/-/blob/affcdeed45755321d7d41c83375729abf9018d5f/src/plasmaquick/dialog.cpp#L1620. Both require that edges are resizable though.
Comment 14 Nate Graham 2022-06-06 17:24:16 UTC
FWIW I have a left side panel, maybe that's relevant?
Comment 15 Luca Carlon 2022-06-06 22:46:24 UTC
Tried it. Also tried to enable "auto hide". Tried also on the secondary screen.
Is this deterministically reproducible for you?
Comment 16 Nate Graham 2022-06-07 00:13:33 UTC
Yes I'm afraid it's 100% reproducible for me.
Comment 17 Luca Carlon 2022-06-07 05:29:13 UTC
And if you created a new user, with a default setup, does it happen there? If it doesn't, then maybe there is a specific setup needed to reproduce it.
Comment 18 Marco Martin 2022-06-07 08:48:23 UTC
i have everyting from today's master, still cannot reproduce
Comment 19 Marco Martin 2022-06-07 08:49:34 UTC
tried to put a panel on the left, still doesn't happen
Comment 20 Bug Janitor Service 2022-06-08 20:43:52 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/545
Comment 21 Nate Graham 2022-06-09 17:17:03 UTC
Git commit 94a0495f137d966442b13d05dbf8c215260b35f5 by Nate Graham, on behalf of Luca Carlon.
Committed on 09/06/2022 at 17:16.
Pushed by ngraham into branch 'master'.

Dialog should use the Dock type unless requested otherwise

When the dialog is requested to be always on top, use the Dock window
type instead of the AppletPopup type unless otherwise requested.

M  +5    -1    src/plasmaquick/dialog.cpp

https://invent.kde.org/frameworks/plasma-framework/commit/94a0495f137d966442b13d05dbf8c215260b35f5