Bug 459249

Summary:	With two systrays on desktop, if one is removed, notifications won't appear next to the remaining one
Product:	[Plasma] plasmashell	Reporter:	Jin Liu <ad.liu.jin>
Component:	System Tray widget	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED FIXED
Severity:	minor	CC:	materka, nate, qydwhotmail
Priority:	NOR	Keywords:	regression
Version First Reported In:	5.25.0
Target Milestone:	1.0
Platform:	Arch Linux
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/commit/a6f719db196ed398da87f3be12040941c74e8700	Version Fixed/Implemented In:	5.26
Sentry Crash Report:

Description Jin Liu 2022-09-17 03:38:13 UTC

SUMMARY
If the desktop has two systrays (e.g. two default panels), then after one of them is removed, new notifications won't appear next to the remaining systray, but at the center of screen.

STEPS TO REPRODUCE
1. Right click on the desktop, add panel, default panel.
2. Right click on the new panel, edit mode, remove panel.

OBSERVED RESULT
Now notifications appear at the center of screen.

EXPECTED RESULT
Notifications appear next to the systray.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.25.90
KDE Frameworks Version: 5.98.0
Qt Version: 5.15.6

Comment 1 Jin Liu 2022-09-17 03:41:53 UTC

Similar problem with taskbar:
1. Add a default panel.
2. Start some applications.
3. Remove the newly-added panel.
4. Running applications still minimize to the non-existing panel.

Comment 2 Fushan Wen 2022-09-17 12:34:47 UTC

Can't reproduce. Perhaps it's already fixed in KF5.99. Need to test again after KF5.99 is released.

Comment 3 Bug Janitor Service 2022-09-17 13:22:53 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/2134

Comment 4 Fushan Wen 2022-09-19 12:29:35 UTC

Git commit 0fb1453f8c131288349d653fa03a4c671f65f803 by Fushan Wen.
Committed on 19/09/2022 at 04:43.
Pushed by fusionfuture into branch 'master'.

applets/notifications: use `Plasmoid.self` in `Globals.adopt`

Plasmoid will become an object in QML after the system tray is deleted,
but we need `!Plasmoid` to return true in order to delete the plasmoid
item in `Globals.plasmoids`, so use `Plasmoid.self` instead.
FIXED-IN: 5.26

M  +3    -1    applets/notifications/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-workspace/commit/0fb1453f8c131288349d653fa03a4c671f65f803

Comment 5 Fushan Wen 2022-09-19 12:32:12 UTC

Git commit a6f719db196ed398da87f3be12040941c74e8700 by Fushan Wen.
Committed on 19/09/2022 at 12:31.
Pushed by fusionfuture into branch 'Plasma/5.26'.

applets/notifications: use `Plasmoid.self` in `Globals.adopt`

Plasmoid will become an object in QML after the system tray is deleted,
but we need `!Plasmoid` to return true in order to delete the plasmoid
item in `Globals.plasmoids`, so use `Plasmoid.self` instead.
FIXED-IN: 5.26
(cherry picked from commit 0fb1453f8c131288349d653fa03a4c671f65f803)

M  +3    -1    applets/notifications/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-workspace/commit/a6f719db196ed398da87f3be12040941c74e8700