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

[todd rme]

On Thu, Sep 23, 2010 at 9:35 AM, Sebastian Kügler <sebas at kde.org> wrote:
> 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/roundedcorn
>> >> 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.cpp
>> >> >   1170001
>> >> >
>> >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/CMakeLis
>> >> > t s.txt PRE-CREATION
>> >> >
>> >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedc
>> >> > o rners.desktop PRE-CREATION
>> >> >
>> >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedc
>> >> > o rners.h PRE-CREATION
>> >> >
>> >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedc
>> >> > o rners.cpp PRE-CREATION
>> >> >
>> >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedc
>> >> > o rners_config.h PRE-CREATION
>> >> >
>> >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedc
>> >> > o rners_config.cpp PRE-CREATION
>> >> >
>> >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedc
>> >> > o rners_config.desktop PRE-CREATION
>> >> >
>> >> > tags/KDE/4.5.1/kdebase/workspace/kwin/effects/roundedcorners/roundedc
>> >> > 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?

Yes, this is a real issue in KDE 4, and has been since the beginning.
There is a workaround in playground utils called noenter that would
prevent your mouse from going off-screen (it wouldn't teleport your
mouse like I think you are saying, the mouse just would not move past
the edge).  However, it doesn't compile on recent versions of KDE, and
it has flaws as well (for instance it doesn't block drag-and-drop
movements from going off-screen).  I think a good solution integrated
into KDE by default is a must-have.

-Todd