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