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