<table><tr><td style="">ngraham 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/D11842">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D11842#241862" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11842#241862</a>, <a href="https://phabricator.kde.org/p/rkflx/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rkflx</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>FWIW, have a +1 for the idea. Removing the categories is not ideal, but having the ratings can be much more enticing for the user to install the app, and there is only so much space… Thus I think it's the better compromise to go with the ratings.</p>

<hr class="remarkup-hr" />

<p>BTW, could not run the patch, because Discover master fails to start for me:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">QQmlApplicationEngine failed to load component
qrc:/qml/DiscoverWindow.qml:159 Cannot assign to non-existent property "showCloseButton"</pre></div>

<p>Any tips? I ran this on a non-master KF5/Plasma desktop, but CMake did not complain so it should be fine, right?</p></div>
</blockquote>

<p>Yeah, we're depending on a feature in the KF 5.45 version of Kirigami, We should mark that dependency.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>One more thing: Lately I'm missing the "before" screenshots in your patches, although they would be really helpful! (If you installed to <tt style="background: #ebebeb; font-size: 13px;">~/usr</tt> instead of <tt style="background: #ebebeb; font-size: 13px;">/usr</tt>, for non-unique apps you could even run both versions in parallel for all your screenshotting needs, or in sequence for unique apps…)</p></blockquote>

<p>Sure, I'll try to get back to including before examples too. It's not technically challenging, I'm just being lazy. :-)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R134 Discover Software Store</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11842">https://phabricator.kde.org/D11842</a></div></div><br /><div><strong>To: </strong>ngraham, apol, Discover Software Store<br /><strong>Cc: </strong>rkflx, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>