broulik added a comment.

  While in principle the patch is fine, the only way this can be null is when 
NotificationPopup.qml (which is a Plasma Dialog) failed to load, so I think 
we're masking a deeper problem here.
  
    var popup = notificationPopupComponent.createObject();
    notificationPositioner.addNotificationPopup(popup);

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D3051

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #plasma, broulik
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas

Reply via email to