| Summary: | With multiple notifications visible, closing the bottom-most one no longer collapses the stack of remaining notifications to fill the empty space | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | slartibart70 |
| Component: | Notifications | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | bizyaev, kde, kde, nate, oleg |
| Priority: | NOR | Keywords: | regression |
| Version First Reported In: | 6.3.3 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/-/commit/c8289fd789d5af909cf748ac926762c090033f8a | Version Fixed In: | |
| Sentry Crash Report: | |||
| Bug Depends on: | |||
| Bug Blocks: | 502160 | ||
| Attachments: | clip | ||
|
Description
slartibart70
2025-01-13 12:42:56 UTC
Any chance you can attach a screen recording that shows this happening? It's rather hard to visualize. Created attachment 177331 [details]
clip
there you are... All i'm doing here is clicking into the popup (which suffices to close it) It's just not visually appealing that popups get 'stuck' up in the list instead of 'falling down'. I never had this in 6.2 Aha! I can reproduce this issue by creating and deleting 5 widgets on the desktop. If I click the close button on the bottom-most one, the stack doesn't "move down" to take over the now free space. Should end up being the same issue. yes, definitely easier than simulating a meeting in teams :-) Can confirm Some notes: * Definitely a 6.3 regression here. * This applies to any persistent notifications, e.g. from Merkuro Calendar or websites in Chrome (I think https://developer.mozilla.org/en-US/docs/Web/API/Notification/requireInteraction is the flag). * Notifications remember that they should reposition when hovered. * The bug has more symptoms: if I get a persistent notification on session startup, it's displayed on the left side of the screen, not the right one where the system tray is, until I move my mouse cursor over it. My guess is that it also jumped before 6.3, but without my interaction. A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5372 Git commit 066d3083c60d1f6d87ada2a60c66ede19c70510d by Marco Martin. Committed on 03/04/2025 at 09:05. Pushed by mart into branch 'master'. appelts/notifications: Fix Notifications positions window position is double buffered, so it's needed to call an explicit QWindow::update() after we ask to move the popup with PlasmaShellWaylandIntegration::get(this)->setPosition(me->pos()); Related: bug 500192, bug 491217, bug 502160 M +1 -0 applets/notifications/notificationwindow.cpp https://invent.kde.org/plasma/plasma-workspace/-/commit/066d3083c60d1f6d87ada2a60c66ede19c70510d Git commit c8289fd789d5af909cf748ac926762c090033f8a by Marco Martin. Committed on 03/04/2025 at 10:38. Pushed by mart into branch 'Plasma/6.3'. appelts/notifications: Fix Notifications positions window position is double buffered, so it's needed to call an explicit QWindow::update() after we ask to move the popup with PlasmaShellWaylandIntegration::get(this)->setPosition(me->pos()); Related: bug 500192, bug 491217, bug 502160 (cherry picked from commit 066d3083c60d1f6d87ada2a60c66ede19c70510d) 066d3083 appelts/notifications: Fix Notifications positions Co-authored-by: Marco Martin <notmart@gmail.com> M +1 -0 applets/notifications/notificationwindow.cpp https://invent.kde.org/plasma/plasma-workspace/-/commit/c8289fd789d5af909cf748ac926762c090033f8a thanks, works again very nicely! |