[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: A KSelectAction dedicated to QTextCodec selection ?
From:       "Robert Knight" <robertknight () gmail ! com>
Date:       2006-09-26 13:43:27
Message-ID: 13ed09c00609260643j52d9f9fi9645ac982bf7401c () mail ! gmail ! com
[Download RAW message or body]

> IMO, a single "Manage..." button would be better, which brings up a
> dialog to add/remove favorite charsets

I second the idea for a single button, but I suggest that it should be
called "Add / Remove..." , for familiarity.

Regards,
Robert.

On 26/09/06, LiuCougar <liucougar@gmail.com> wrote:
> On 9/25/06, Michel Hermier <michel.hermier@gmail.com> wrote:
> > Hi,
> > 
> > I just commited the code available here:
> > http://websvn.kde.org/trunk/KDE/kdenetwork/kopete/protocols/irc/ui/kcodecaction.cpp?rev=588417&view=auto
> >  http://websvn.kde.org/trunk/KDE/kdenetwork/kopete/protocols/irc/ui/kcodecaction.h?rev=588417&view=auto
> >  http://websvn.kde.org/trunk/KDE/kdenetwork/kopete/protocols/irc/ui/kcodecactiontest.cpp?rev=588417&view=auto
> >  http://websvn.kde.org/trunk/KDE/kdenetwork/kopete/protocols/irc/ui/kcodecactiontest.h?rev=588417&view=auto
> >  
> > The class is named KCodecAction (for historical reasons), but maybe a
> > better name would be KTextCodecAction or KCharsetsAction. Also
> > included a test code, so that it can be tested.
> > 
> > I was thinking instead of adding an Add/Remove button,
> IMO, a single "Manage..." button would be better, which brings up a
> dialog to add/remove favorite charsets
> 
> Regards,
> Cougar
> 
> > maybe we can
> > add some auto hidden fields (based on date of last clicks and stored
> > in a KConfig), and add a "More ..." action class at the end of the
> > list, if not all the elements are shown ?
> > 
> > Michel
> > 
> 
> 
> --
> "People's characters are strengthened through struggle against
> difficulties; they are weakened by comfort."
> - Old Chinese adage
> 


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic