Bug 493737 - Some notifications are distorted and missaligned
Summary: Some notifications are distorted and missaligned
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (other bugs)
Version First Reported In: master
Platform: Other Linux
: VHI normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2024-09-27 15:03 UTC by Gurenko Alex
Modified: 2024-10-23 18:42 UTC (History)
8 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 6.3.0
Sentry Crash Report:


Attachments
Originally appeared notification (46.03 KB, image/png)
2024-09-27 15:03 UTC, Gurenko Alex
Details
Notification after the click (48.60 KB, image/png)
2024-09-27 15:04 UTC, Gurenko Alex
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gurenko Alex 2024-09-27 15:03:58 UTC
Created attachment 174141 [details]
Originally appeared notification

SUMMARY

Fedora 41 beta just got plasma 6.1.90 and notifications occasionally misbehave

STEPS TO REPRODUCE

Don't really have a steps to reproduce, but get notifications

OBSERVED RESULT

Some (small number of notifications) contain artifacts (white or black portions, scrambled text or moved outside of the screen). Upon clicking on it, it restores the view but moves a little to much on the left (not same alignment as "normal" notification)

EXPECTED RESULT

All notifications appear in a same spot and without artifacts

SOFTWARE/OS VERSIONS

Operating System: Fedora Linux 41
KDE Plasma Version: 6.1.90
KDE Frameworks Version: 6.6.0
Qt Version: 6.7.2
Kernel Version: 6.11.0-63.fc41.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 24 × AMD Ryzen 9 5900X 12-Core Processor
Memory: 31.3 GiB of RAM
Graphics Processor: AMD Radeon RX 7900 XTX
Manufacturer: Micro-Star International Co., Ltd.
Product Name: MS-7C84
System Version: 1.0

ADDITIONAL INFORMATION
Comment 1 Gurenko Alex 2024-09-27 15:04:30 UTC
Created attachment 174142 [details]
Notification after the click

This is how it moves after the original is clicked
Comment 2 Gurenko Alex 2024-09-27 16:23:08 UTC
Actually scratch the spacing issue, this seems to be new in 6.1.90 overall,  "Notification after the click" is how the "normal" notifications are shown now
Comment 3 Nate Graham 2024-09-27 18:20:36 UTC
Very strange. We did change Notification layout for 6.1.90 which may have triggered this.
Comment 4 Nate Graham 2024-09-29 00:35:33 UTC
I suspect this may be a different manifestation of Bug 492540, or at least it might be fixed if we revert the changes that caused it.

I haven't seen this particular issue myself, though. Would be good to find a concrete way to reproduce it.
Comment 5 Gurenko Alex 2024-09-29 17:00:51 UTC
(In reply to Nate Graham from comment #4)
> I suspect this may be a different manifestation of Bug 492540, or at least
> it might be fixed if we revert the changes that caused it.
> 
> I haven't seen this particular issue myself, though. Would be good to find a
> concrete way to reproduce it.

I'm using Breeze Light mode, so not sure if that's it. I've tried to spam the test notifications, but those work very consistent, I think it has something to do with styling, when non-text elements are present (like Google Calendar has it's picture embedded), but that's speculation on my side
Comment 6 David Edmundson 2024-09-30 15:36:05 UTC
This is a good trigger:

notify-send "$(date)" 'asdfadfsdfsdfadfasdfqw asdfasdfasdfasd'

you need it to wrap. 

Underlying distortion is a QtWayland issue - we're calling QWaylandEGLWindow::updateSurface on both the main thread (resizing) and the render thread (showing the first frame), it's guarded to not crash, but depending on concurrency the oldest size might finish last. 

Trigger is notification layouting doing something super weird, it's resizing after rendering the first frame, which is wrong. We should fix both, priority on a notification fix as that can land in 6.2.0
Comment 7 Bug Janitor Service 2024-10-01 08:21:17 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/4772
Comment 8 Nate Graham 2024-10-02 14:06:51 UTC
Fixed in 6.2.0 with a mass revert of the recent changes to Notifications, to reduce risk. Still an issue on master.
Comment 9 Nate Graham 2024-10-23 18:42:11 UTC
And now fixed in master for the next release with https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/4791.