mart added inline comments.

INLINE COMMENTS

> AbstractCard.qml:31
> +        id: bg
> +        readonly property color pressedColor: 
> Qt.tint(Kirigami.Theme.backgroundColor, Qt.rgba(
> +                                                        
> Kirigami.Theme.highlightColor.r,

as we have a ColorUtils class now, since i've seen this super long thing just 
to make a color translucent many things, is maybe worth to have a 
ColorUtils.opacity(Kirigami.Theme.highlightColor, 0.3) instead?

REPOSITORY
  R169 Kirigami

BRANCH
  card_shadowrect

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

To: ahiemstra, #kirigami, #vdg, cblack
Cc: mart, ngraham, nicolasfella, cblack, plasma-devel, fbampaloukas, GB_2, 
domson, dkardarakos, apol, ahiemstra, davidedmundson

Reply via email to