<table><tr><td style="">broulik updated this revision to Diff 62540.<br />broulik edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-kfcqot32ceto7a3/">(Show Details)</a><br />broulik 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/D22735">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Emit dataChanged before inserting and only update the to be signalled range</li>
<li class="remarkup-list-item">Add some docs to make future me happy</li>
<li class="remarkup-list-item">Fix current index behavior with mouse and keyboard</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">This adds a new `Milou.ResultsModel` for runner results which tries very hard to avoid model resets and instead signals addition, removal, and changes in results.<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> One of the main advantages is that now the highlighted item isn't reset every time the results change, i.e. results that come in delayed won't unexpectedly change the keyboard selection.<br />
</span><br />
The various result processing, filtering, and grouping features are split into tiny dedicated proxy models to make the code easier to follow and maintain. The QML API isn't changed so that look and feel themes don't need to be adjusted.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R112 Milou</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D22735?vs=62527&id=62540">https://phabricator.kde.org/D22735?vs=62527&id=62540</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22735">https://phabricator.kde.org/D22735</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
lib/CMakeLists.txt<br />
lib/qml/ResultDelegate.qml<br />
lib/qml/ResultsView.qml<br />
lib/qml/qmlplugins.cpp<br />
lib/resultsmodel.cpp<br />
lib/resultsmodel.h<br />
lib/runnerresultsmodel.cpp<br />
lib/runnerresultsmodel.h</div></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>davidedmundson, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>