Commit 3609f66a authored by Bhushan Shah's avatar Bhushan Shah 📱

notifications: on mobile align popup to top center

any other position doesn't really make sense on mobile, so ignore popup location

Reviewers: broulik

Reviewed By: broulik

Subscribers: plasma-devel

Tags: #plasma

Differential Revision:
parent 7f2ae105
......@@ -26,6 +26,7 @@ import org.kde.plasma.plasmoid 2.0
import org.kde.plasma.core 2.0 as PlasmaCore
import org.kde.plasma.components 2.0 as Components
import org.kde.kquickcontrolsaddons 2.0
import org.kde.kirigami 2.11 as Kirigami
import org.kde.notificationmanager 1.0 as NotificationManager
import org.kde.taskmanager 0.1 as TaskManager
......@@ -90,6 +91,11 @@ QtObject {
property var plasmoids: []
property int popupLocation: {
// if we are on mobile, we can ignore the settings totally and just
// align it to top left
if (Kirigami.Settings.isMobile) {
return Qt.AlignTop | Qt.AlignHCenter;
switch (notificationSettings.popupPosition) {
// Auto-determine location based on plasmoid location
case NotificationManager.Settings.CloseToWidget:
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment