<table><tr><td style="">davidedmundson added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22256">View Revision</a></tr></table><br /><div><div><p>The fundamental problem with this approach is you only sort through the dirs correctly, but not the total across multiple paths.</p>

<p>i.e<br />
If I have two paths dirA and dirB and want things sorted by date this patch will have all the images in dirA sorted by date, followed by all the images in dirB sorted by date.</p>

<p>So for those reasons I think the other patch addresses works a bit better handling it dynamically, also allowing us to get rid of one of the model instances.</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22256">https://phabricator.kde.org/D22256</a></div></div><br /><div><strong>To: </strong>msdobrescu, ngraham, davidre, Plasma<br /><strong>Cc: </strong>davidedmundson, davidre, ngraham, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>