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

Reply via email to