davidedmundson added a comment.

  Visual indication on mouseover is generally a good thing ++

INLINE COMMENTS

> ActionButton.qml:44
> +        Behavior on opacity {
> +            PropertyAnimation { // OpacityAnimator makes it turn black at 
> random intervals
> +                duration: units.longDuration * 2

I ran with:

  opacity: activeFocus || containsMouse ? 1 : 0.85
      Behavior on opacity {
          OpacityAnimator { // OpacityAnimator makes it turn black at random 
intervals
              duration: units.longDuration * 2
              easing.type: Easing.InOutQuad
          }
  }

on my system, worked fine. Can you give any more details on "randomly"

> ActionButton.qml:55
> +        height: width
> +        radius: width
> +        color: PlasmaCore.ColorScope.textColor

width/2

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

To: filipf, rooty, ngraham, #vdg, #plasma
Cc: davidedmundson, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to