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