Bug 491217 - If widget removal undo notification was moved to avoid overlapping panel configuration, it does not restore itself to the correct location after clicking "Exit edit mode" button
Summary: If widget removal undo notification was moved to avoid overlapping panel conf...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (show other bugs)
Version: 6.1.3
Platform: Arch Linux Linux
: NOR minor
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks: 502160
  Show dependency tree / graph
 
Reported: 2024-08-03 13:45 UTC by Patrick Silva
Modified: 2025-04-03 10:58 UTC (History)
2 users (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/plasma-workspace/-/commit/c8289fd789d5af909cf748ac926762c090033f8a
Version Fixed In:
Sentry Crash Report:

Attachments
screen recording (2.39 MB, video/webm)
2024-08-03 13:45 UTC, Patrick Silva 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2024-08-03 13:45:16 UTC 
Created attachment 172245 [details]
screen recording

STEPS TO REPRODUCE
1. right-click on panel and choose "Show panel configuration"
2. hover over a widget placed on the panel and choose 'Remove" - Plasma shows a notification with "Undo" button above  "Panel settings" menu
3. leave the edit mode by clicking on "Exit edit mode" button

OBSERVED RESULT
after the last step, the "Undo" notification remains in the same position. It moves itself to lower right corner when clicked.
Please watch the attached screen recording.

EXPECTED RESULT
"Undo" notification should move itself to lower right corner immediately after leaving the edit mode

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 6.1.3
KDE Frameworks Version: 6.4.0
Qt Version: 6.7.2
Graphics Platform: Wayland
Comment 1 Patrick Silva 2024-08-03 13:50:20 UTC 
Can also reproduce by removing a widget on desktop in the step 2.

The bug does not occur if I leave the edit mode by clicking on "Done" button.
Comment 2 Nate Graham 2024-08-05 18:28:21 UTC 
Can reproduce.
Comment 3 Bug Janitor Service 2025-04-03 09:05:32 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5372
Comment 4 Marco Martin 2025-04-03 09:33:22 UTC 
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 498599, bug 502160

M  +1    -0    applets/notifications/notificationwindow.cpp

https://invent.kde.org/plasma/plasma-workspace/-/commit/066d3083c60d1f6d87ada2a60c66ede19c70510d
Comment 5 Marco Martin 2025-04-03 10:58:52 UTC 
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 498599, 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