| Summary: | Ability to change position in which the notification is displayed. | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Stephan Henrichs <stiepen22> |
| Component: | Notifications | Assignee: | 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: | http://commits.kde.org/plasma-workspace/9429c845ae3fee1d65b026b2d496aae1305db232 | Version Fixed In: | 5.3.0 |
| Sentry Crash Report: | |||
|
Description
Stephan Henrichs
2015-03-04 17:33:39 UTC
For clarification: I want an option to change the location. I do not want you to change it for everyone. 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/ 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 Thats some great news! Thanks a lot :) (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 Ideally it would just follow the primary screen setting which is set elsewhere. But that's for a new report ;) Is it possible this feature was removed again with 5.11? |