Bug 427685

Summary: Notification window closes when going back if no other window is visible
Product: [Plasma] plasmashell Reporter: rLy <rly07>
Component: System TrayAssignee: Konrad Materka <materka>
Status: RESOLVED FIXED    
Severity: normal CC: akozlovskiy119, kinofhek, materka, med.medin.2014, rly07, stefanomilani92
Priority: NOR    
Version: master   
Target Milestone: 1.0   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In: 5.21
Sentry Crash Report:
Attachments: video of bug

Description rLy 2020-10-14 10:35:42 UTC
Pressing the back button when selecting one of the app (KDE Connect for ie.) closes the whole notification when no other window displayed. If any other window displayed then pressing the back button properly brings back the "menu".

STEPS TO REPRODUCE
1. Have no other window displayed
2. Bring up the notifications and select an app
3. Press back button on the top left

OBSERVED RESULT
Closes the whole notifications window

EXPECTED RESULT
Go back to the menu to select something else

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Arch linux
KDE Plasma Version: 5.20.0
KDE Frameworks Version: 5.75.0
Qt Version: 5.15.1
Comment 1 Konrad Materka 2020-10-16 16:35:28 UTC
Hi, can you add some screenshots? I do not understand the issue fully.
It is possible that we have a fix already, waiting for review, please check description of:
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/350
Comment 2 rLy 2020-10-16 18:13:35 UTC
Created attachment 132435 [details]
video of bug
Comment 3 rLy 2020-10-16 18:19:10 UTC
(In reply to Konrad Materka from comment #1)
> Hi, can you add some screenshots? I do not understand the issue fully.
> It is possible that we have a fix already, waiting for review, please check
> description of:
> https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/350

I attached a recording of it. Looking at the description of that merge it might fix the original issue. But I also noticed one more which can be also seen on the video. Randomly a new window shows up in the task manager called Plasma when going back to the grid. Most of the times it goes away immediately but sometimes it sticks until the dialog is closed.
Comment 4 Konrad Materka 2020-10-20 20:27:13 UTC
*** Bug 427881 has been marked as a duplicate of this bug. ***
Comment 5 medin 2020-11-05 10:53:29 UTC
I have also the same problem on Manjaro KDE. When an app is opened and I click on back button in tray window the focus is stolen from tray window to the app opened in the back, then tray window recaptures focus again. It's like the whole tray windows is closed and reopened when back button is clicked.

Operating System: Manjaro Linux
KDE Plasma Version: 5.20.2
KDE Frameworks Version: 5.75.0
Qt Version: 5.15.1
Kernel Version: 5.4.74-1-MANJARO
OS Type: 64-bit
Comment 6 Konrad Materka 2020-11-07 12:44:22 UTC
Git commit 04112536133184eeb53af6016e9aeddf9db546d4 by Konrad Materka.
Committed on 07/11/2020 at 12:03.
Pushed by kmaterka into branch 'master'.

[applets/systemtray] Rework of "expanded" state handling

Refactor and simply logic of "expanded" state when dialog with active applet or list of hidden applets is shown.

Fixes a problem with keyboard shortcut activation.
It also fixes a bug when dialog closes on "go-back" button click.
Related: bug 400278
FIXED-IN: 5.21

M  +2    -3    applets/systemtray/container/package/contents/ui/main.qml
M  +20   -2    applets/systemtray/package/contents/ui/CurrentItemHighLight.qml
M  +8    -18   applets/systemtray/package/contents/ui/ExpandedRepresentation.qml
M  +6    -7    applets/systemtray/package/contents/ui/ExpanderArrow.qml
M  +1    -1    applets/systemtray/package/contents/ui/HiddenItemsView.qml
M  +3    -3    applets/systemtray/package/contents/ui/PlasmoidPopupsContainer.qml
A  +93   -0    applets/systemtray/package/contents/ui/SystemTrayState.qml     [License: LGPL (v2+)]
M  +1    -1    applets/systemtray/package/contents/ui/items/AbstractItem.qml
M  +4    -12   applets/systemtray/package/contents/ui/items/PlasmoidItem.qml
M  +11   -52   applets/systemtray/package/contents/ui/main.qml
M  +12   -0    applets/systemtray/systemtray.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/04112536133184eeb53af6016e9aeddf9db546d4
Comment 7 Konrad Materka 2021-01-12 19:52:28 UTC
*** Bug 431353 has been marked as a duplicate of this bug. ***
Comment 8 Nate Graham 2021-02-03 19:54:42 UTC
*** Bug 432447 has been marked as a duplicate of this bug. ***