[Differential] [Commented On] D2144: Fix warnings
mart (Marco Martin)
noreply at phabricator.kde.org
Tue Jul 12 13:49:52 UTC 2016
mart added inline comments.
INLINE COMMENTS
> ContextIcon.qml:29
> height: Units.iconSizes.smallMedium
> - property real morph: root.position
> + property real morph: 0
> property color color: Theme.textColor
does the context menu icon keeps morphing correctly with this?
> PageRow.qml:326
>
> - if (itemX >= contentX && itemX + mainFlickable.currentItem.width <= contentX + mainFlickable.width) {
> + if (itemX >= contentX && mainFlickable.currentItem && itemX + mainFlickable.currentItem.width <= contentX + mainFlickable.width) {
> return;
why adding check over mainFlickable.currentItem ? doesn't guard anything
REPOSITORY
rKIRIGAMI Kirigami
REVISION DETAIL
https://phabricator.kde.org/D2144
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: apol, #kirigami, mart
Cc: plasma-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160712/ead3f598/attachment.html>
More information about the Plasma-devel
mailing list