Bug 427685 - Notification window closes when going back if no other window is visible
Summary: Notification window closes when going back if no other window is visible
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Tray (show other bugs)
Version: master
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: Konrad Materka
URL:
Keywords:
: 427881 431353 432447 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-10-14 10:35 UTC by rLy
Modified: 2021-02-03 19:54 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.21


Attachments
video of bug (1.18 MB, video/webm)
2020-10-16 18:13 UTC, rLy
Details

Note You need to log in before you can comment on or make changes to this 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. ***