Bug 500192 - Low battery notification takes noticeably long to go on top of other widgets
Summary: Low battery notification takes noticeably long to go on top of other widgets
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (show other bugs)
Version: 6.3.0
Platform: Arch Linux Linux
: NOR minor
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks: 502160
  Show dependency treegraph
 
Reported: 2025-02-16 15:13 UTC by Fernando M. Muniz
Modified: 2025-04-03 10:59 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Not very responsive, could be more smooth. (3.02 MB, video/x-matroska)
2025-02-16 15:13 UTC, Fernando M. Muniz
Details
The command didn't work. (53.17 KB, image/png)
2025-02-18 18:57 UTC, Fernando M. Muniz
Details
Slightly laggy. (3.31 MB, video/mp4)
2025-02-18 19:10 UTC, Fernando M. Muniz
Details
Low battery and explosion notification at the same time. (3.89 MB, video/x-matroska)
2025-02-28 19:24 UTC, Fernando M. Muniz
Details
It behaves normally when using the low battery logo while charged ( By using Bug 500838 ) (3.17 MB, video/x-matroska)
2025-02-28 19:51 UTC, Fernando M. Muniz
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Fernando M. Muniz 2025-02-16 15:13:51 UTC
Created attachment 178441 [details]
Not very responsive, could be more smooth.

This might happen with other notifications as well, I haven't tested.
Comment 1 Nate Graham 2025-02-18 15:54:17 UTC
Can you test?
Comment 2 Fernando M. Muniz 2025-02-18 16:43:09 UTC
(In reply to Nate Graham from comment #1)
> Can you test?

I've tested with https://www.bennish.net/web-notifications.html it teleported up and down without delay.
Comment 3 Nate Graham 2025-02-18 18:55:32 UTC
Can you run 

notify-send --urgency=critical --app-name "System Exploder Monitor" --icon granatier "Urgent Alert" "Your computer is about to explode"

In a terminal window? If you do, does this notification also try to dodge the System Tray like the critical battery notification one does? And if it does, is it slow like the critical battery notification is?
Comment 4 Fernando M. Muniz 2025-02-18 18:57:14 UTC
Created attachment 178537 [details]
The command didn't work.
Comment 5 Nate Graham 2025-02-18 19:06:12 UTC
You didn't run the entire command. Just copy-and paste the entire line of text into a terminal window and hit Enter.
Comment 6 Fernando M. Muniz 2025-02-18 19:10:37 UTC
Created attachment 178540 [details]
Slightly laggy.

For some reason I didn't copy the entire command.
Comment 7 Nate Graham 2025-02-18 19:27:28 UTC
Can you please stop changing the assignee value of bug reports? Just leave it alone.

The speed looks pretty much expected in your latest screen recording. I can't think of a reason why only the low battery notification could be affected here.
Comment 8 Fernando M. Muniz 2025-02-28 19:24:57 UTC
Created attachment 178979 [details]
Low battery and explosion notification at the same time.
Comment 9 Fernando M. Muniz 2025-02-28 19:51:26 UTC
Created attachment 178983 [details]
It behaves normally when using the low battery logo while charged ( By using Bug 500838 )

Looks like it's not related to the notification, but the low battery itself feeding less.
Comment 10 Bug Janitor Service 2025-04-03 09:05:29 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5372
Comment 11 Marco Martin 2025-04-03 09:33:14 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 498599, bug 491217, bug 502160

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

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