<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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/D24721">View Revision</a></tr></table><br /><div><div><p>I like the idea behind this but I'm not sure this is the right way to do it. Instead of changing the strings in the backend, we should add the new text (plus icons, preferably) on the UI side, in the QML files. See <tt style="background: #ebebeb; font-size: 13px;">discover/qml/ApplicationDelegate.qml</tt></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/D24721">https://phabricator.kde.org/D24721</a></div></div><br /><div><strong>To: </strong>brunogoncalves, Discover Software Store, ngraham<br /><strong>Cc: </strong>plasma-devel, Discover Software Store, LeGast00n, The-Feren-OS-Dev, semareit, jraleigh, bam, fbampaloukas, GB_2, ragreen, ixoos, ZrenBot, James, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>