<table><tr><td style="">gregormi 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/D13706">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/D13706#282765" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D13706#282765</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I'd like to see the missing AppStream URLs filled in for all the software in this list. There are currently some claring omissions, especially for KDE software (e.g. ksysguard) for which there is definitely an appstream ID available). For any software that doesn't have any AppStream information (and is therefore not visible or installable via Discover), I would actually advocate removing it from the list, and making the presence of AppStream data a pre-condition of inclusion--the reason being that otherwise an <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Install</span></span></span> button can't be presented to the user. Entries without an <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Install</span></span></span> button are just frustrating, tantamount to taunting them ("here's some cool software you could use; oh, sorry, can't actually install it lol")</p></div>
</blockquote>

<p>Hi Nate, thanks for reviewing.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">I tried <tt style="background: #ebebeb; font-size: 13px;">appstreamcli ksysguard</tt> and <tt style="background: #ebebeb; font-size: 13px;">appstreamcli ding</tt>. Both do not yield a result. => I vote not to remove a program from this list just because it has no obvious appstream id (yet).</li>
<li class="remarkup-list-item">Programs which serve the same purpose should have a comment why they are in the list, like it was done with giggle ("// good for searching in history"). In such cases, for me it would be ok if there is no appstream id yet.</li>
<li class="remarkup-list-item">For me, it would be ok, if Nicolas does not have to search for all the missing appstream ids - except if you have fun to do it, Nicolas :-). The patch is a definite improvement to the current state and missing ids could also be added later incrementally.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13706">https://phabricator.kde.org/D13706</a></div></div><br /><div><strong>To: </strong>nicolasfella, Frameworks, gregormi, ngraham<br /><strong>Cc: </strong>ngraham, dhaumann, kde-frameworks-devel, michaelh, bruns<br /></div>