D29335: Implement notification grouping on Android

This revision was automatically updated to reflect the committed changes. Closed by commit R289:942bddded289: Implement notification grouping on Android (authored by vkrause). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D29335?vs=81731=83063#toc REPOSITORY R289 KNotifications

D29335: Implement notification grouping on Android

vkrause updated this revision to Diff 81731. vkrause added a comment. Replace the simple ref count with a full child id tracking. The ref count got out of sync when an existing notification is updated, using a set fixes that. REPOSITORY R289 KNotifications CHANGES SINCE LAST UPDATE

D29335: Implement notification grouping on Android

vkrause added a comment. Still not good enough, updating existing notfication messes up the refcounter, resulting still in leftover group elements. REPOSITORY R289 KNotifications BRANCH grouping REVISION DETAIL https://phabricator.kde.org/D29335 To: vkrause, nicolasfella Cc:

D29335: Implement notification grouping on Android

vkrause updated this revision to Diff 81725. vkrause added a comment. Explicitly track if notification groups are still in use. This fixes group summaries staying active when we explicitly close a notification, rather then having the user or system dismiss it. REPOSITORY R289

D29335: Implement notification grouping on Android

vkrause added a comment. This isn't good to go yet, there are corner cases where the group summary item stays around after closing the last notification, working on fixing this. REPOSITORY R289 KNotifications BRANCH master REVISION DETAIL https://phabricator.kde.org/D29335 To:

D29335: Implement notification grouping on Android

nicolasfella added inline comments. INLINE COMMENTS > vkrause wrote in NotifyByAndroid.java:171 > That seems counter-productive to me, as the Android API documentation always > speaks about API level numbers, so you'd need to do an additional translation > to/from the letters in your head. So

D29335: Implement notification grouping on Android

vkrause added inline comments. INLINE COMMENTS > nicolasfella wrote in NotifyByAndroid.java:171 > Please use > https://developer.android.com/reference/android/os/Build.VERSION_CODES > instead of hardcoding numbers That seems counter-productive to me, as the Android API documentation always

D29335: Implement notification grouping on Android

nicolasfella accepted this revision. nicolasfella added a comment. This revision is now accepted and ready to land. Haven't tested it, but the code looks sensible INLINE COMMENTS > NotifyByAndroid.java:171 > +Notification.Builder builder; > +if (Build.VERSION.SDK_INT >= 26) {

D29335: Implement notification grouping on Android

vkrause added a comment. Collapsed: F8276057: Screenshot_20200501_161952.PNG Expanded: F8276059: Screenshot_20200501_162043.PNG REPOSITORY R289 KNotifications REVISION DETAIL

D29335: Implement notification grouping on Android

vkrause created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. vkrause requested review of this revision. REVISION SUMMARY This is available starting at API level 20, which is below our minimal requirement. Grouping can be disabled by the