rename kdeui/utils/KPalette

Robert Knight robertknight at gmail.com
Mon Jun 11 21:53:46 BST 2007


Matthew,

While you are at it, could you rename the KPalette's nrColors() method
to something better, eg. colorCount() or just count()

Regards,
Robert.

On 11/06/07, Robert Knight <robertknight at gmail.com> wrote:
> Good with me.  With the new color roles added to kdelibs Konsole might
> not need the class currently called KPalette.
>
> Regards,
> Robert.
>
> On 11/06/07, Aaron J. Seigo <aseigo at kde.org> wrote:
> > On Monday 11 June 2007, Matthew Woehlke wrote:
> > > The current KPalettte (kdelibs/kdeui/util) "is totally unrelated to
> > > QPalette." It is used in kdeui/color, by Konsole, and (IIRC) according
> > > to aseigo by one lone app in extragear.
> > >
> > > If there are no objections/better ideas I would like to rename this to
> > > KIndexedPalette, which seems to be a more appropriate name (other
> > > suggestions welcome), next Monday.
> >
> > i support this change, yes. particularly if we are going to have a class
> > that
> > more closely resembles QPalette (at least in purpose) being introduced.
> >
> > p.s. be sure to update kdelibs/KDE4PORTING.html when you do this
> >
> > --
> > Aaron J. Seigo
> > humru othro a kohnu se
> > GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43
> >
> > Full time KDE developer sponsored by Trolltech (http://www.trolltech.com)
> >
>




More information about the kde-core-devel mailing list