<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/D12285">View Revision</a></tr></table><br /><div><div><p>Discover gets a lot of criticism for the header image too. Same complaint: it's a pointless waste of space, an image there for the sake of being there. I really don't think this pattern works at all and would advocate removing it from Kirigami entirely. These images are just wasted space, and no matter how pretty they are, that won't change.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12285">https://phabricator.kde.org/D12285</a></div></div><br /><div><strong>To: </strong>nicolasfella, KDE Connect, VDG<br /><strong>Cc: </strong>ngraham, anemeth, albertvaka, abetts, kdeconnect, mtijink, VDG, KDE Connect, skymoore, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara, apol<br /></div>