D29340: fix(kcm): avoid recursive OpacityMask which leads to teardown crash

2020-05-02 Thread David Edmundson
davidedmundson added inline comments. INLINE COMMENTS > romangg wrote in Output.qml:60 > Is the change making this unnecessary? I.e. do the graphics still look > "smooth"? Default is true so it makes no difference. There's generally no reason to set it to false. REPOSITORY R104 KScreen REV

D29340: fix(kcm): avoid recursive OpacityMask which leads to teardown crash

2020-05-02 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R104:cfd95e6bff4b: fix(kcm): avoid recursive OpacityMask which leads to teardown crash (authored by davidedmundson). REPOSITORY R104 KScreen CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29340

D29340: fix(kcm): avoid recursive OpacityMask which leads to teardown crash

2020-05-01 Thread Roman Gilg
romangg accepted this revision. romangg added a comment. This revision is now accepted and ready to land. > Docs explicitly say this. When you reference some "docs" please take 20 seconds and directly link it. Every piece you write here and in git commit messages is potentially read by h

D29340: fix(kcm): avoid recursive OpacityMask which leads to teardown crash

2020-05-01 Thread David Edmundson
davidedmundson created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. davidedmundson requested review of this revision. REVISION SUMMARY In the current code OpacityMask's source is the parent which contains itself, which doesn't make sense. Docs explici