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

[Asraniel]

If it is a bug or not i don't know, but this is the behaviour. Really annoying 
if you have the panel on the screen with the smaller resolution, because you 
can go beyond the panel, breaking your workflow (you can't quickly click on 
kickoff by throwing your mouse to the bottom lef).

Beat Wolf

Am Donnerstag 23 September 2010, um 15.35:50 schrieb Sebastian Kügler:
> On Wednesday, September 22, 2010 19:02:14 todd rme wrote:
> > On Tue, Sep 21, 2010 at 6:05 PM, Nuno Pinheiro <nuno at oxygen-icons.org>
> 
> wrote:
> > > A Terça, 21 de Setembro de 2010 21:41:33 Martin Gräßlin você escreveu:
> > >> -----------------------------------------------------------
> > >> This is an automatically generated e-mail. To reply, visit:
> > >> http://svn.reviewboard.kde.org/r/5225/#review7713
> > >> -----------------------------------------------------------
> > >> 
> > >> 
> > >> I am still rather opposed to include this effect in a default
> > >> workspace setup. I really like the work you are putting into it, but
> > >> I think that this effect is hardly used by anyone and is not
> > >> something that belongs into a default setup. And we should not
> > >> include everything and the kitchensink because we can do it
> > >> (therefore we have Compiz, see
> > >> http://smspillaz.wordpress.com/2010/09/15/philosophy/ ). I am also
> > >> unhappy with some of the effects we currently ship (including snow,
> > >> which was written by me), so this is more a general thing. I think
> > >> such effects should go to either kdeartwork, kdeplasma-addons or
> > >> extragear. Till we have found a nice solution I would recommend to
> > >> publish on kde-look/apps (although that is also not so nice as we
> > >> tend to break the ABI).
> > >> 
> > >> 
> > >> tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedco
> > >> rn ers .cpp <http://svn.reviewboard.kde.org/r/5225/#comment7832>
> > >> 
> > >>     This check looks superfluous. In line 35 you set glTexture to 0,
> > >>     so
> > >> 
> > >> for both xrender and opengl the pointer is at least 0. As deleting
> > >> null pointers is save you can get rid of the if statement and I think
> > >> also of the #ifdef.
> > >> 
> > >> 
> > >> - Martin
> > >> 
> > >> 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.cp
> > >> >   p 1170001
> > >> > 
> > >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/CMakeLi
> > >> > s t s.txt PRE-CREATION
> > >> > 
> > >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/rounded
> > >> > c o rners.desktop PRE-CREATION
> > >> > 
> > >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/rounded
> > >> > c o rners.h PRE-CREATION
> > >> > 
> > >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/rounded
> > >> > c o rners.cpp PRE-CREATION
> > >> > 
> > >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/rounded
> > >> > c o rners_config.h PRE-CREATION
> > >> > 
> > >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/rounded
> > >> > c o rners_config.cpp PRE-CREATION
> > >> > 
> > >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/rounded
> > >> > c o rners_config.desktop PRE-CREATION
> > >> > 
> > >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/rounded
> > >> > c o rners_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
> > > 
> > > from a design prespective this efect is
> > > 1 noncence, removes the hit area on the edges of the scren.
> > > 2 not good enough, (if you would want to do it you should do it
> > > properlie via some shape distrotion mesh and some global lights and
> > > shadows)
> > > 
> > > so IMO it should not be in defoult land
> > > oxygen guy, "I make the pretty pictures"
> > 
> > A bit off topic, but is this able to prevent the mouse from entering
> > certain portions of the screen?  If so, then could the technique be
> > used to prevent the mouse from entering invisible portions of the
> > screen when using two monitors with different resolutions?  Currently
> > it is easy for the mouse to get lost there, and it makes things more
> > difficult for unhiding panels.  Although there is a valid question
> > regarding whether rounded corners belongs in kwin, I think preventing
> > the mouse from going off-screen (from the user point of view) is
> > definitely worthwhile for inclusion in kwin.
> 
> I've not used such a setup with kwin4, but had two monitors of different
> resolution with kwin3, and the position of the cursor would just be changed
> when it would slip offscreen. It was just not possible for the mouse cursor
> to go offscreen.
> 
> Has this behaviour (in X, I suppose) changed? Are you talking of a real
> existing bug?