Bug 493737

Summary: Some notifications are distorted and missaligned
Product: [Plasma] plasmashell Reporter: Gurenko Alex <agurenko>
Component: NotificationsAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, kde, kde, kdedev, laurent.rineau, nate, notmart, postix
Priority: VHI Keywords: regression
Version First Reported In: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=492540
https://bugs.kde.org/show_bug.cgi?id=493803
Latest Commit: Version Fixed/Implemented In: 6.3.0
Sentry Crash Report:
Attachments: Originally appeared notification
Notification after the click

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.