[compiz] patch for colored drop-shadow

[Guillaume Seguin]

Judgind by the users feedback after QuinnStorm included this patch in her
CVS, it looks like some users find this option useful since they can have
borders and shadows of the same color, or shadows that fit their
background... (for example some beautiful orange shadows with default Ubuntu
Dapper theme).

Regards

Guillaume Seguin

2006/5/3, David Reveman <davidr at novell.com>:
>
> On Fri, 2006-04-28 at 15:01 -0400, Quinn Storm wrote:
> > On Fri, 28 Apr 2006 13:16:22 +0200
> > David Reveman <davidr at novell.com> wrote:
> >
> > > On Tue, 2006-04-25 at 20:28 +0200, Mirco Müller wrote:
> > > > Greetings everybody!
> > > >
> > > >   Here's a patch (against compiz from CVS-head about 30 min.
> > > > ago) that adds a shadow_color option to the parameters of the
> > > > decoration-plugin. I only needed to add a few lines to decoration.c
> > > > and gnome-window-decorator.c. The only thing missing is the
> > > > compiz.schema. I dare not touch that XML-file manually. So after
> > > > you applied the patch, recompiled compiz and restarted g-w-d and
> > > > compiz you'll have to fire up gconf-editor and add a
> > > > string-parameter with the name shadow_color yourself.
> > > >
> > > >   You can use the normal HTML-like syntax for defining the
> > > > wanted color:
> > > >
> > > >   #fff or #ffffff -> white
> > > >   #f00 or #ff0000 -> red
> > > >   #f73 or #ff7733 -> orange
> > >
> > >
> > > Patch looks OK, I'll add it if people think this functionality is
> > > useful. I don't want to add options just because we can.
> > >
> > > >
> >
> > Options are just that, options.  If you don't want to use them there's
> > no reason to, but they are there if you do.  If you are concerned with
> > the user's experience, they'll see everything through a UI anyway, and
> > if you want to hide options there, that's fine.
> >
> > I don't see any reason -not- to add an option if it doesn't impact
> > things like speed, functionality, etc. in a negative way.
>
> We can expose add all kinds of useless crap through options if we want.
> If no one uses an option except for when trying what it does, then it's
> useless. I don't want useless options. Every option adds some extra code
> and a few strings, might be very little code in some cases but still
> unnecessary code that I don't want.
>
> Things that it's obvious that people want to adjust should of course be
> exposed through options if possible. For other things, I think it's best
> to wait until someone request them before they are added.
>
> To me, the shadow color is not an obvious thing that people want to
> adjust. Hence, why I didn't add it in the first place. But if people are
> requesting this option, I'll gladly accept macslow's patch.
>
> -David
>
> _______________________________________________
> compiz mailing list
> compiz at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/compiz
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.freedesktop.org/archives/compiz/attachments/20060508/e3163042/attachment.htm