> On Feb. 19, 2014, 11:06 a.m., Aleix Pol Gonzalez wrote: > > src/knotificationmanager.cpp, line 180 > > <https://git.reviewboard.kde.org/r/115695/diff/3/?file=243841#file243841line180> > > > > ? > > Martin Klapetek wrote: > Yet once again the description xD - "it's full of ... FIXMEs to indicate > what needs doing" > > Aleix Pol Gonzalez wrote: > Still you shouldn't ship such comments, so it's probably why nobody > checked the "ship it" thing.
I'm not expecting the ship is as much just yet, but rather comments on how the new architecture without knotify looks & works. I put it up early on as Kevin wanted to have this in Alpha2, which is in 10 days from now. So I wanted to get comments on the general approach asap so I could still manage to handle any wanted changes in the arch before Alpha2. - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115695/#review50197 ----------------------------------------------------------- On Feb. 13, 2014, 11:14 a.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115695/ > ----------------------------------------------------------- > > (Updated Feb. 13, 2014, 11:14 a.m.) > > > Review request for kde-workspace, KDE Frameworks, Plasma, and Sune Vuorela. > > > Repository: knotifications > > > Description > ------- > > This patch merges KNotify daemon into KNotificationManager to have less > daemons running and less dbus traffic. The patch is not yet finished (and for > now it's full of QDebugs, that will all be removed and FIXMEs to indicate > what needs doing), but as the Alpha2 is quite soon, I'd like to start the > general review asap so some more changes can be done if needed. > > Now it's KNotificationManager that handles the KNotifyPlugin-s and hands them > the notification directly. KNotifyConfig was repurposed a bit, now it serves > mostly just as the .notifyrc wrapper, all the rest is reused from the > KNotification object. There are some changes in the KNotification API - id() > and appName() are now exposed to public and "slotReceivedId(int)" is now also > public so that KNotificationManager can directly give it an id. I'd like to > rename this and make it a non-slot. It's not the DBus/Galago server ID > anymore, that is handled in NotifyByPopup which is responsible for > communicating with the galago server (all the methods there were renamed to > actually have *galago* in the name so it's clear), therefore the mapping of > DBus/Galago Server ids is managed only there as it is actually only needed > here. KNotitification::id() is assigned by the KNotificationManager and it's > a simple increasing counter. > > The UI/Plasmoid changes will come next - basically the plan is to put only > the "Persistent" notifications in the notifications history. > > > Diffs > ----- > > src/knotifyconfig.h PRE-CREATION > src/knotifyconfig.cpp PRE-CREATION > src/knotifyplugin.h PRE-CREATION > src/knotifyplugin.cpp PRE-CREATION > src/notifybypopup.h PRE-CREATION > src/notifybypopup.cpp PRE-CREATION > src/notifybypopupgrowl.h PRE-CREATION > src/notifybypopupgrowl.cpp PRE-CREATION > CMakeLists.txt 63ebf71 > src/CMakeLists.txt a81b913 > src/knotification.h 00554ba > src/knotification.cpp 5d7405b > src/knotificationmanager.cpp a4d0dfa > src/knotificationmanager_p.h 81d962d > > Diff: https://git.reviewboard.kde.org/r/115695/diff/ > > > Testing > ------- > > Works perfectly with both plasma notifications and kpassivepopup. > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel