----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115695/#review50197 -----------------------------------------------------------
src/knotification.cpp <https://git.reviewboard.kde.org/r/115695/#comment35297> remove qDebugs? src/knotification.cpp <https://git.reviewboard.kde.org/r/115695/#comment35298> Improve message and make qWarning? src/knotification.cpp <https://git.reviewboard.kde.org/r/115695/#comment35299> Needed? src/knotificationmanager.cpp <https://git.reviewboard.kde.org/r/115695/#comment35300> This will be done in the future? Maybe it would be better to commit when there are no regressions? src/knotificationmanager.cpp <https://git.reviewboard.kde.org/r/115695/#comment35301> qWarning? src/knotificationmanager.cpp <https://git.reviewboard.kde.org/r/115695/#comment35302> ? src/notifybypopupgrowl.h <https://git.reviewboard.kde.org/r/115695/#comment35303> Is growl still the thing for MacOS X? I'm sorry I cannot give better insight about the moving the daemon here. It makes sense to me, but I have no idea why it used to be this way. - Aleix Pol Gonzalez On Feb. 13, 2014, 10: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, 10: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
