<table><tr><td style="">mickaelbo updated this revision to Diff 60908.<br />mickaelbo 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/D22059">View Revision</a></tr></table><br /><div><div><p>This is the patch corresponding to the third suggestion screenshot.<br />
I also modified the search.ui file in order to change some QPushButton to QToolButton. This way, all "icon only" buttons have the same look and it allows to hide the other buttons text (this was used in the second suggestion).</p></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D22059?vs=60534&id=60908">https://phabricator.kde.org/D22059?vs=60534&id=60908</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22059">https://phabricator.kde.org/D22059</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>addons/search/plugin_search.cpp<br />
addons/search/plugin_search.h<br />
addons/search/search.ui</div></div></div><br /><div><strong>To: </strong>mickaelbo, Kate, sars, VDG, ngraham<br /><strong>Cc: </strong>ngraham, kwrite-devel, cblack, arvidhansson, ian, jguidon, hannahk, Ghost6, jraleigh, fbampaloukas, squeakypancakes, alexde, IohannesPetros, GB_2, trickyricky26, mglb, domson, michaelh, crozbo, ndavis, firef, demsking, skadinna, aaronhoneycutt, cullmann, sars, dhaumann, mbohlender<br /></div>