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

List:       kde-panel-devel
Subject:    Re: knotificationitem: api prettiness
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2009-05-27 19:48:28
Message-ID: 200905271348.33194.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Wednesday 27 May 2009, Richard Moore wrote:
> On Wed, May 27, 2009 at 12:27 PM, Aaron J. Seigo <aseigo@kde.org> wrote:
> > On Tuesday 26 May 2009, Marco Martin wrote:
> >> 1) we have two way to set icons: by name and by pixmap, right now by
> >> name is setIcon()
> >> by pixmap is setImage()
> >> (that should reflect in dbus too)
> >> that looks a bit weird, would be better maybe setIconName() and
> >> setIcon()?
> >
> > another proposal:
> >
> > setIconByName
> > setIconByPixmap
> >
> > that way we don't end up with a setIcon(QIcon) that the compiler would
> > use silently when someone ports from KSystemTray to KNotificationItem (we
> > really want to discourage use of the setIcon(QIcon) method) and it says
> > exactly what it's doing.
>
> This is a kind of +0.5. I can see the setters, but iconByName() and
> iconByPixmap() seem a bit odd.

iconName()
iconPixmap()

yes, not perfectly symmetric.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software


["signature.asc" (application/pgp-signature)]

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


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