D28306: Make the hover state clarifying, but not main

David Edmundson noreply at phabricator.kde.org
Thu Mar 26 14:31:44 GMT 2020


davidedmundson added a comment.


  @ngraham the default theme doesn't change with this patch.
  
  I believe there are no plans to do so.

INLINE COMMENTS

> Task.qml:331
> +            if (isHovered)
> +                return TaskTools.taskPrefix(basePrefix + "-hover").concat(TaskTools.taskPrefix("hover")).concat(TaskTools.taskPrefix(basePrefix));
> +            return TaskTools.taskPrefix(basePrefix);

this is a bit longwinded, can we make a function in TasksTools?

REPOSITORY
  R119 Plasma Desktop

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

To: epopov, davidedmundson, #vdg
Cc: ngraham, epopov, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200326/88133c28/attachment.html>


More information about the Plasma-devel mailing list