On user notifications...
martin.klapetek at gmail.com
Tue May 10 13:05:13 CEST 2011
2011/5/9 Aaron J. Seigo <aseigo at kde.org>
> On Saturday, May 7, 2011 14:13:04 Martin Klapetek wrote:
> > > in any case, the fix would be pretty simple. the code is in kde-
> > > runtime/knotify/notifybypopup.cpp .. at a quick glance it looks like
> > > simply checking for an icon on the KNotifyConfig* that is handed in to
> > > the notify method would suffice. perhaps this could even be done in the
> > > getAppCaptionAndIconName method.
> > Great, I'll have a look. Would this patch be accepted for 4.7 or is it
> > already too late?
> probably too late for the 4.7 schedule, but we can put into git at least
> it will be ready for merging. if it was a smaller feature set i might
> we try and cheat a bit since we aren't at hard freeze yet, but it's a
> significant change that will affect public api in kdelibs, knotify and
> eventually a key plasmoid :)
Ok, I think we can live with this for a while. I'll work on this later this
week and then I'll post a patch to reviewboard. Is that ok with you?
> 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 Development Frameworks
> KDE-Telepathy mailing list
> KDE-Telepathy at kde.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Plasma-devel