[Differential] [Commented On] D2984: Center the notification label on Breeze's lockscreen and make it use the full width before wrapping

[davidedmundson (David Edmundson)]

davidedmundson added a comment.


  Generally good, just needs some a bit of tidying up
  
  Also do you have commit access?

INLINE COMMENTS

> SessionManagementScreen.qml:89-90
>              id: notificationsLabel
>              Layout.maximumWidth: units.gridUnit * 16
>              Layout.alignment: Qt.AlignHCenter
> +            Layout.fillWidth: true

Then this line is redundant

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

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

To: pritambaral, #plasma
Cc: davidedmundson, jensreuterberg, pritambaral, plasma-devel, lesliezhai, ali-mohamed, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161009/e3c48aaf/attachment.html>