[prev in list] [next in list] [prev in thread] [next in thread]
List: kwin
Subject: Re: [PATCH] Configuration option for window caption drawing in
From: Lubos Lunak <l.lunak () suse ! cz>
Date: 2008-04-02 13:04:56
Message-ID: 200804021504.56500.l.lunak () suse ! cz
[Download RAW message or body]
On Saturday 29 of March 2008, Louai Al-Khanji wrote:
> On Fri, Mar 21, 2008 at 3:54 PM, Lubos Lunak <l.lunak@suse.cz> wrote:
> > On Thursday 20 of March 2008, Louai Al-Khanji wrote:
> > > Hi,
> > >
> > > I started to find the captions visually jarring, so I wrote a patch to
> > > add a config option for them. They are off by default in the patch, I
> > > can flip this around if people think they should be on by default.
> >
> > They should be.
> >
> > > Please comment if you don't like this. :)
> >
> > Hmm. I wonder if the option should not get more specific and apply to
> > the icons as well?
>
> For now I commited the previous patch with the text shown by default.
> It makes sense to me to make the icons configurable as well, I'll add
> an option for that.
>
> Do you have a preference on the config ui? I'm a bit undecided between
> a checkbox for each the text and the icon or a combobox with different
> combinations. I think the checkboxes might make more sense.
By 'more specific' I didn't mean 'yet another option'. I can see that
somebody perhaps doesn't like the captions over the whole window, but I don't
see how somebody would want the caption and not the icon. So I thought your
option should perhaps apply to the icon as well. But, thinking of it, it
should probably stay as it is.
--
Lubos Lunak
KDE developer
--------------------------------------------------------------
SUSE LINUX, s.r.o. e-mail: l.lunak@suse.cz , l.lunak@kde.org
Lihovarska 1060/12 tel: +420 284 028 972
190 00 Prague 9 fax: +420 284 028 951
Czech Republic http://www.suse.cz
_______________________________________________
Kwin mailing list
Kwin@kde.org
https://mail.kde.org/mailman/listinfo/kwin
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic