Review Request 127427: [Fading Node] Don't mix opacity separately

[Kai Uwe Broulik]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127427/
-----------------------------------------------------------

(Updated March 20, 2016, 10:36 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit 92e2c07adfd75f1f9e5a4a0bc592c9799ba20867 by Kai Uwe Broulik to branch master.


Bugs: 355894
    http://bugs.kde.org/show_bug.cgi?id=355894


Repository: plasma-framework


Description
-------

This breaks icon item fading when opacity is < 1 as can be observed in ksmserver's logout dialog


Diffs
-----

  src/declarativeimports/core/fadingnode.cpp 64bf3ec 

Diff: https://git.reviewboard.kde.org/r/127427/diff/


Testing
-------

Works fine now, tests still pass, verified that it doesn't flicker anymore:

PlasmaCore.IconItem {
    width: 200; height: 200
    active: mouse.containsMouse
    source: "folder-home"
    opacity: 0.5
    
    MouseArea {
        id: mouse
        anchors.fill: parent
        hoverEnabled: true
    }
}


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160320/fcb0d875/attachment.html>