Bug 344841

Summary: Ability to change position in which the notification is displayed.
Product: [Plasma] plasmashell Reporter: Stephan Henrichs <stiepen22>
Component: NotificationsAssignee: Martin Klapetek <mklapetek>
Status: RESOLVED FIXED    
Severity: wishlist CC: commander.alchemy, kde.org, kdelibs-bugs, simonandric5
Priority: NOR    
Version: 5.2.0   
Target Milestone: 1.0   
Platform: unspecified   
OS: All   
Latest Commit: Version Fixed In: 5.3.0

Description Stephan Henrichs 2015-03-04 17:33:39 UTC
Beginning with Plasma 5 it is very tempting for me to put notifications in top right. Not knowing how i'd do that i headed to the IRC channel. After some fiddeling around we came to the conclusion that it was not implemented, and it was suggested to me to report it here. In case it is already implemented, please make it more obvious how to do that.

Reproducible: Always
Comment 1 Stephan Henrichs 2015-03-04 18:13:50 UTC
For clarification: I want an option to change the location. I do not want you to change it for everyone.
Comment 2 Martin Klapetek 2015-03-24 17:26:37 UTC
It's not yet implemented in any released code, but there is a patch under review now at https://git.reviewboard.kde.org/r/122842/
Comment 3 Martin Klapetek 2015-04-06 18:09:23 UTC
Git commit 9429c845ae3fee1d65b026b2d496aae1305db232 by Martin Klapetek.
Committed on 06/04/2015 at 18:09.
Pushed by mklapetek into branch 'master'.

Allow selecting notifications position on screen

FIXED-IN: 5.3.0
REVIEW: 122842

M  +2    -1    applets/notifications/CMakeLists.txt
A  +13   -0    applets/notifications/lib/CMakeLists.txt
C  +37   -25   applets/notifications/lib/notificationsapplet.cpp [from: applets/notifications/package/contents/ui/configNotifications.qml - 050% similarity]
C  +31   -26   applets/notifications/lib/notificationsapplet.h [from: applets/notifications/package/contents/ui/configNotifications.qml - 054% similarity]
M  +11   -1    applets/notifications/package/contents/ui/Notifications.qml
A  +267  -0    applets/notifications/package/contents/ui/ScreenPositionSelector.qml     [License: GPL (v2/3)]
M  +23   -1    applets/notifications/package/contents/ui/configNotifications.qml
M  +1    -0    applets/notifications/package/metadata.desktop
M  +19   -6    applets/notifications/plugin/notificationshelper.cpp
M  +17   -3    applets/notifications/plugin/notificationshelper.h

http://commits.kde.org/plasma-workspace/9429c845ae3fee1d65b026b2d496aae1305db232
Comment 4 Stephan Henrichs 2015-04-06 18:16:13 UTC
Thats some great news! Thanks a lot :)
Comment 5 Artur O. 2015-09-03 20:42:30 UTC
(In reply to Martin Klapetek from comment #2)
> It's not yet implemented in any released code, but there is a patch under
> review now at https://git.reviewboard.kde.org/r/122842/

Would it be possible to also select monitor? 
https://bugs.kde.org/show_bug.cgi?id=348608
Comment 6 Martin Klapetek 2015-09-04 14:37:26 UTC
Ideally it would just follow the primary screen setting which is set elsewhere. But that's for a new report ;)
Comment 7 dns2utf8 2018-02-07 09:45:05 UTC
Is it possible this feature was removed again with 5.11?