Review Request 126459: Add a hint to skip notifications grouping

[Kai Uwe Broulik]

-----------------------------------------------------------
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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151229/b831f2ec/attachment.html>