<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/D20693">View Revision</a></tr></table><br /><div><div><p>To show that a thumbnail clickable, switching to the pointing hand cursor when hovering over a thumbnail could work.</p>

<p>However I notice that the actual list delegates in the browse view seem to add frames and shadows to the thumbnails there, and they look okay. The frame's proportions even perfectly match the aspect ratio of the thumbnail: <a href="https://phabricator.kde.org/F6788649" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6788649: Screenshot_20190423_074013.png</a></p>

<p>Why doesn't any of that work here?</p>

<p>Aesthetics-wise, I'm okay deleting the shadow, but I'm less thrilled about also deleting the frame surrounding the image. Without that, the images look naked in the view, like they're just floating there, disconnected from everything: <a href="https://phabricator.kde.org/F6788651" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6788651: Screenshot_20190423_074053.png</a></p>

<p>Rewriting this in QML would be lovely (not in this patch though, obviously). I think we have a mockup of a new UI for it somewhere which I can't find right now but I'll try to dig it up.</p></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/D20693">https://phabricator.kde.org/D20693</a></div></div><br /><div><strong>To: </strong>leinir, KNewStuff, ngraham, sitter, VDG<br /><strong>Cc: </strong>ndavis, kde-frameworks-devel, KNewStuff, michaelh, ZrenBot, ngraham, bruns<br /></div>