Review Request: new kwin effect: roundedcorners - make corners of the desktop rounded

[Thomas Lübking]

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



tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedcorners.cpp
<http://svn.reviewboard.kde.org/r/5225/#comment7830>

    there'd be no point in duping the picture, one could just requse the same over and over again and just reset the transformation matrix, but ...



tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedcorners.cpp
<http://svn.reviewboard.kde.org/r/5225/#comment7831>

    Unlike with OpenGL, matrix transforamtions can be very slow with various XRender implemententations.
    
    Consider to generate a minimal empty circle (ie. 2r x 2r, all black, inner circle is transparent) and dump out the required sections from this instead (since this is easy and fast with xrender - other than with open gl ;-)


- Thomas


On 2010-09-21 18:34:04, Christoph Fritz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5225/
> -----------------------------------------------------------
> 
> (Updated 2010-09-21 18:34:04)
> 
> 
> Review request for kwin, Plasma, Martin Gräßlin, and Thomas Lübking.
> 
> 
> Summary
> -------
> 
> Inspired by roundedge http://www.nongnu.org/roundedge/ this kwin effect makes corners of the desktop rounded.
> Older Macs and Monitors had this feature too.
> 
> 
> Diffs
> -----
> 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/CMakeLists.txt 1170001 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/configs_builtins.cpp 1170001 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/CMakeLists.txt PRE-CREATION 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedcorners.desktop PRE-CREATION 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedcorners.h PRE-CREATION 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedcorners.cpp PRE-CREATION 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedcorners_config.h PRE-CREATION 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedcorners_config.cpp PRE-CREATION 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedcorners_config.desktop PRE-CREATION 
>   tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedcorners_config.ui PRE-CREATION 
> 
> Diff: http://svn.reviewboard.kde.org/r/5225/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> roundedcorners_without_frame
>   http://svn.reviewboard.kde.org/r/5225/s/498/
> with_simulated_border
>   http://svn.reviewboard.kde.org/r/5225/s/499/
> 
> 
> Thanks,
> 
> Christoph
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100921/96bdb899/attachment.htm