ngraham added inline comments.

INLINE COMMENTS

> main.qml:141
> +        QQC2.Button {
> +            Layout.alignment: Qt.AlignRight
> +

Align left to match the emerging standard for other {nad Add new thing} buttons

> user_manager.desktop:9
> +X-KDE-Weight=40
> +X-KDE-System-Settings-Parent-Category=personalization
> +Name=Users

Should be `accountdetails`, or else the Account Details group will have only 
KWallet in it and no will longer need to exist, so then KWallet would also need 
to be moved up a level

REPOSITORY
  R119 Plasma Desktop

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

To: cblack, #plasma, #vdg, ngraham
Cc: yurchor, iasensio, meven, crossi, The-Feren-OS-Dev, davidedmundson, 
broulik, filipf, ngraham, nicolasfella, zzag, plasma-devel, Orage, LeGast00n, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to