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

[Christoph Fritz]

> On 2010-09-01 16:42:41, Martin Gräßlin wrote:
> > To be honest I do not see the usefulness of such an effect :-)
> > 
> > If I understand your code correctly you are creating four GL Textures for the corners. That are three too many. Mirroring textures is very easy in OpenGL. In general I would prefer if such eyecandy stuff only uses GLSL (which raises the requirements to OpenGL 2). So you would just specify one VBO and use vertex attributes to indicate the corner and do the mirroring in the fragment shader. This would also add the possibility to easily do nice radiant fade outs, so that only the pixel nearest to the corner is black while the other parts are translucent.
> > 
> > It would be nice if you could attach a screenshot and please get feedback from Oxygen (designers) and please add Plasma to the groups as well. This is heavily influencing the look of the Plasma Workspace, so they should also have their say on it.
> > 
> > Oh and please have a look on the HACKING file in kwin source directory for coding style.
> 
> Christoph Feck wrote:
>     As far as I understand it, this should not be enabled as default, but only enable eye candy to all those that want emulate Mac designs (and I can already see all the screen shots posted to sites everywhere, where this "effect" is enabled).
>     
>     Cannot comment on the technical side, but smooth anti-aliasing is a MUST (standard OpenGL anti-aliasing probably is too coarse).

Thanks a lot for reviews and comments.

Can't upload a screenshot in this reviewboard here: [Errno 13] Permission denied: '/var/www/sites/svn.reviewboard.kde.org/htdocs/media/uploaded/images/2010/09/02' 

So I added one to this place:
http://img842.imageshack.us/img842/6695/withdarkgreyborder.jpg (with a dark grey frame to simulate a monitor and to get a better impression)

plasma-group is added.

As I can see from  http://www.oxygen-icons.org/?page_id=16  Thomas Lübking is already from the Oxygen-Team.


- Christoph


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


On 2010-09-02 10:01:55, Christoph Fritz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5225/
> -----------------------------------------------------------
> 
> (Updated 2010-09-02 10:01:55)
> 
> 
> Review request for kwin, Plasma and Martin Gräßlin.
> 
> 
> 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
> -------
> 
> 
> Thanks,
> 
> Christoph
> 
>

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