<div dir="ltr">Hey,<div><br></div><div>so as the patch on RB was apparently too big to review in 14 days, I tried to divide it in smaller commits and pushed into a branch mklapetek/knotify-merge in kde:knotifications.</div>

<div><br></div><div>So please have a look at that, the commitdiffs might be a bit confusing as it was created from the big diff, but I tried explaining it in the commit message. Also the code has a lot of comments, so you can also just go over the code - it's only three files - knotification.cpp (400 LOC), knotificationmanager.cpp (200 LOC) and notifybypopup.cpp (760 LOC).</div>

<div><br></div><div>I've been testing this for about a week now and it all works nicely. So at this point I'd like to get this into the alpha2 release and keep improving it afterwards, the public API shouldn't change anymore (that's KNotification only anyway).</div>

<div><br></div><div>As I'd really like to get this in alpha2, I'm going to merge this by Friday night if there will be no hard objections.<br clear="all"><div><br></div><div>Cheers</div>-- <br><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div>


</div></div>