Bug 344841 - Ability to change position in which the notification is displayed.
Summary: Ability to change position in which the notification is displayed.
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (show other bugs)
Version: 5.2.0
Platform: unspecified All
: NOR wishlist
Target Milestone: 1.0
Assignee: Martin Klapetek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-03-04 17:33 UTC by Stephan Henrichs
Modified: 2018-02-07 09:45 UTC (History)
4 users (show)

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


Attachments

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