[Differential] [Commented On] D4214: Make sure we display enabled ListItem on hover

[David Edmundson]

davidedmundson added a comment.


  Concept seems fine. New binding has some typos

INLINE COMMENTS

> ListItem.qml:97
> +        prefix: (listItem.sectionDelegate ? "section" :
> +                (background.pressed || listItem.checked || listItem.containsMouse) ? "pressed" : "normal")
>  

why are you now following containsMouse. That's a behavioural change

How can background.pressed be true. I don't see where it exists?
do you mean itemMouse.pressed?

REPOSITORY
  R242 Plasma Framework (Library)

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

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

To: apol, #plasma
Cc: davidedmundson, plasma-devel, #frameworks, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170120/d3ee2b57/attachment.html>