Review Request: Plasma::WindowEffects functions to render window thumbnails

Aaron Seigo aseigo at kde.org
Wed Oct 7 03:21:02 CEST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1800/#review2572
-----------------------------------------------------------


great to see this code moving into Plasma::WindowEffects. (maybe we can convince a compiz'r to provide a compiz implementation of this as well... though common xatoms would be even better of course.) a few small comments; other than the enum vs string thing, it looks good to my eyes.


/trunk/KDE/kdelibs/plasma/windoweffects.h
<http://reviewboard.kde.org/r/1800/#comment1895>

    QString or an enumeration? an enum would give us compiler checking on spelling :)



/trunk/KDE/kdelibs/plasma/windoweffects.cpp
<http://reviewboard.kde.org/r/1800/#comment1896>

    this really feels like something that should be found in KWindowSystem; though i'm not sure if Seli would be cool with a convenience function only we are using? can't hurt to ask, anyways.


- Aaron


On 2009-10-06 21:10:44, Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1800/
> -----------------------------------------------------------
> 
> (Updated 2009-10-06 21:10:44)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> this is the tiny part for the overview i had promised (that could or could not be used, anyways..)
> those are some functions in Plasma::WindowEffects to abstract away some of the code to do the taskbar thumbnails, so it would be handy to use it also from somewhere else without having to directly use X calls
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/plasma/private/windowpreview.cpp 1032016 
>   /trunk/KDE/kdelibs/plasma/private/windowpreview_p.h 1032016 
>   /trunk/KDE/kdelibs/plasma/windoweffects.h 1032016 
>   /trunk/KDE/kdelibs/plasma/windoweffects.cpp 1032016 
> 
> Diff: http://reviewboard.kde.org/r/1800/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco
> 
>



More information about the Plasma-devel mailing list