Bug 366360 - Notifications popup is so large it cannot be closed
Summary: Notifications popup is so large it cannot be closed
Status: RESOLVED DUPLICATE of bug 361389
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (other bugs)
Version First Reported In: 5.7.2
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: 1.0
Assignee: Martin Klapetek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-02 20:52 UTC by Till Schäfer
Modified: 2016-08-03 13:31 UTC (History)
3 users (show)

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

Attachments
screenshot of the problem (398.41 KB, image/png)
2016-08-02 20:53 UTC, Till Schäfer 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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)