[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