graesslin requested changes to this revision.
graesslin added a reviewer: graesslin.
graesslin added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> LockScreen.qml:32
>      property UserSelect userSelect: null
> -    property int interfaceVersion: 
> org_kde_plasma_screenlocker_greeter_interfaceVersion ? 
> org_kde_plasma_screenlocker_greeter_interfaceVersion : 0
> -    signal clearPassword()
> +    property int interfaceVersion: 
> org_kde_plasma_screenlocker_greeter_interfaceVersion
>  

this change I don't understand. It's not described in the change.

> LockScreen.qml:59-63
> -    Component.onCompleted: {
> -        if (root.interfaceVersion < 2) {
> -            mainLoader.source = "LockScreenUi.qml";
> -        }
> -    }

why is that removed?

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

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

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

Reply via email to