<table><tr><td style="">ngraham accepted this revision.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D18043">View Revision</a></tr></table><br /><div><div><p>I think this is fine. The search field doesn't have to specify what it's going to be searching in if it's totally obvious, as it is here. This also makes it consistent with other similar instances of list-with-search-field-on-top, as in the Effects KCM.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R295 KCMUtils</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18043">https://phabricator.kde.org/D18043</a></div></div><br /><div><strong>To: </strong>rooty, VDG, Plasma, kde-frameworks-devel, ngraham<br /><strong>Cc: </strong>ngraham, pino, shubham, kde-frameworks-devel, michaelh, bruns<br /></div>