<table><tr><td style="">leinir created this revision.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />leinir requested review of this revision.
</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/D29447">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This patch primarily fixes the issue that when picking the "Updateable Only" option on NewStuff.Page, we would be shown all the entries anyway. It also fixes duplicated entries on the "Installed Only" option, and further adds a little placeholder for when the view is expectedly empty.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Use the right connections for update entry display</li>
<li class="remarkup-list-item">While potentially expensive, only add entries to the model once</li>
<li class="remarkup-list-item">Use Kirigami.PlaceholderMessage to show we have no entries when we don't</li>
</ul>

<p>BUG:416762</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Without this patch, behaviour is as shown in <a href="https://bugs.kde.org/show_bug.cgi?id=416762" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=416762</a></p>

<p>With this patch, the behaviour is as expected (shows only updateable entries with that option selected, only shows installed entries once with that option, and shows a useful message when the list is empty and expected to be empty)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>BRANCH</strong><div><div>fix-show-only-updates (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29447">https://phabricator.kde.org/D29447</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/core/itemsmodel.cpp<br />
src/qtquick/qml/Page.qml<br />
src/qtquick/quickitemsmodel.cpp</div></div></div><br /><div><strong>To: </strong>leinir<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>