D12281: Enable (de)selecting all apps in notification filter

[Nicolas Fella]

nicolasfella added inline comments.

INLINE COMMENTS

> mtijink wrote in NotificationFilterActivity.java:203
> Why in a thread? The previous code didn't do that.

The database operation is quite expensive. Previously only one item at a time was updated so it was only very short. Now all items are updated at once resulting in a noticeable lag when done in the UI thread

REPOSITORY
  R225 KDE Connect - Android application

REVISION DETAIL
  https://phabricator.kde.org/D12281

To: nicolasfella, #kde_connect, broulik
Cc: mtijink, broulik, #kde_connect, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, ahmedbesbes, ndavis, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, ach, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20180417/7c72535c/attachment.html>