----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126459/#review90310 -----------------------------------------------------------
src/knotification.h (line 251) <https://git.reviewboard.kde.org/r/126459/#comment61799> This must be 0x10! 0x10 = 0b10000 (one bit set) 0x22 = 0b10110 (multiple bits) You unintentionally set a bunch of other flags with it, you probably didn't notice that you set the PersistentFlag as the Plasma delete notification sets it anyway. :) - Kai Uwe Broulik On Dez. 29, 2015, 8:07 nachm., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126459/ > ----------------------------------------------------------- > > (Updated Dez. 29, 2015, 8:07 nachm.) > > > Review request for KDE Frameworks. > > > Repository: knotifications > > > Description > ------- > > Plasma currently groups notifications that have the same > notification title or if they are on a "blacklist" (like > media player apps that popup a notification for each song > change), but sometimes the grouping needs to be skipped. > This is for example when user removes two widgets from his > desktop and the "Undo" notification appears, these cannot > be grouped. > > This is currently handled rather by a hack in the notifications > server, so introducing this new flag should allow for a clean > and proper solution. > > > Diffs > ----- > > src/knotification.h 3a4acd5 > src/notifybypopup.cpp e824d39 > > Diff: https://git.reviewboard.kde.org/r/126459/diff/ > > > Testing > ------- > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
