davidedmundson added inline comments.

INLINE COMMENTS

> mart wrote in KeyboardButton.qml:13
> is this domain valid for sddm as well?

It was...though that broke when this folder was moved. Good spot.

> mart wrote in UserDelegate.qml:85
> this version you have is probably is significantly faster (and in this case 
> i'll shut up :p)
> 
> but wouldn't look cleaner to have something along the lines:
> 
> layer.enabled: true
> 
>   layer.effect: OpacityMask {
>       maskSource: Rectangle {
>           anchors.centerIn: parent
>           width: imageWrapper.width
>           height: width
>           radius: width/2
>       }
>   }

It's a bit more complex with the white frame that has to be transparent on the 
inside.

So that then needs to be drawn somehow. I guess making an SVG of a circle would 
work.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

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

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

Reply via email to