[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: knotificationitem: api prettiness
From: Richard Moore <richmoore44 () gmail ! com>
Date: 2009-05-27 19:35:38
Message-ID: 5491a5150905271235u5496f598h1a7c1224c38c586e () mail ! gmail ! com
[Download RAW message or body]
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.
Rich.
_______________________________________________
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]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic