Bug 366360

Summary:	Notifications popup is so large it cannot be closed
Product:	[Plasma] plasmashell	Reporter:	Till Schäfer <till2.schaefer>
Component:	Notifications	Assignee:	Martin Klapetek <mklapetek>
Status:	RESOLVED DUPLICATE
Severity:	normal	CC:	kde, kde, plasma-bugs-null
Priority:	NOR
Version First Reported In:	5.7.2
Target Milestone:	1.0
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	screenshot of the problem

Description Till Schäfer 2016-08-02 20:52:20 UTC

Notification Popups can be larger than the screen. In this case the close button is out of the screen (and clickable) are. If the notification is scheduled to stay. There is no way to get rid of the notification, than re-logging in. 

I have attached an example screenshot with a popup message from kmail.

Solution: If the notification is to large, it should add a scroll pane or something similar.

Reproducible: Always

Comment 1 Till Schäfer 2016-08-02 20:53:36 UTC

Created attachment 100421 [details]
screenshot of the problem

Comment 2 Kai Uwe Broulik 2016-08-03 12:38:43 UTC


*** This bug has been marked as a duplicate of bug 361389 ***

Comment 3 Till Schäfer 2016-08-03 12:50:50 UTC

Isn't this bug more generic? I think the notification framework should take care of such situations and it should not rely on sane implementations of each individual application. The application developer might also not be able to know the screen resolution and the size and visual representation settings (e.g. font size) for the notification area -> i.e. the amount of information, which can be presented at a single message.

=> I would like to see a scrollable message window in this case.

Comment 4 Till Schäfer 2016-08-03 12:53:49 UTC

ups, i just saw that the proposed solution is also generic in the other bug.. sorry for the noise

*** This bug has been marked as a duplicate of bug 361389 ***

Comment 5 Kai Uwe Broulik 2016-08-03 12:54:55 UTC

;)

I also made a patch which unfortunately missed the 5.7 feature freeze, testing welcome: https://phabricator.kde.org/D1479

Comment 6 Till Schäfer 2016-08-03 13:31:56 UTC

i will test this in a few hours (i am currently updating to 5.7.3 anyway an will apply the path during that upgrade)