D8165: Added Auto-Expand Folder On Unread option

Kevin Tardif noreply at phabricator.kde.org
Sat Oct 7 17:14:46 BST 2017


sagara added inline comments.

INLINE COMMENTS

> dvratil wrote in subscriptionlistview.cpp:438
> Shouldn't this call `setExpanded(current, true)`? The condition implies that the `current` index has unread (sub)feeds and thus should be expanded, no?

Yes, I moved the statement out to keep the indentation flat during cleanup but seems I forgot to negate the condition when doing so. At a quick glance the patch seemed to work alright so I didn't pick up on it.

REPOSITORY
  R201 Akregator

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

To: sagara, dvratil
Cc: dvratil, #kde_pim, dvasin, winterz, vkrause, mlaurent, knauss
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20171007/ab97e07f/attachment.html>


More information about the kde-pim mailing list