Review Request 126475: Make sure PlasmaCore.WindowThumbnail doesn't redirect windows while disabled or invisible

Eike Hein hein at kde.org
Tue Dec 22 18:27:47 UTC 2015


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

(Updated Dec. 22, 2015, 6:27 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma, David Edmundson and Martin Gräßlin.


Changes
-------

Submitted with commit c64a94a265acd5003fbbb4e3abc7fdb72726b4c3 by Eike Hein to branch master.


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


Repository: plasma-framework


Description
-------

PlasmaCore.WindowThumbnail acts as a mini-compositor and redirects windows to paint a thumbnail of them. Hovering a Task Manager button delegate will down the line instanciate some WindowThumbnails for the tooltip, and these stick around when the tooltip is hidden. This causes windows to keep being redirected, e.g. severely breaking game performance after hovering them in the Task Manager (see associated bug).

I initially tried just wiping the thumbnail delegates on the Task Manager side when the tooltip goes hidden, but this breaks UX performance when moving across tasks in the applet because the tooltip delegate's visibility prop cycles between true and false from one ToolTipArea to the next. I'll still be tweaking the Task Manager code to at the very least do this when tooltips are disabled to avoid even instanciating, but in the meantime, making WindowThumbnail more idiot-proof strikes me as a good idea regardless.


Diffs
-----

  src/declarativeimports/core/windowthumbnail.cpp 5fc5c5e 

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


Testing
-------


Thanks,

Eike Hein

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


More information about the Plasma-devel mailing list