<table><tr><td style="">kossebau 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/D7061" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D7061#131226" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D7061#131226</a>, <a href="https://phabricator.kde.org/p/igorkushnir/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@igorkushnir</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>There is a similar simpler review request here: <a href="https://git.reviewboard.kde.org/r/126856/diff/3#index_header" class="remarkup-link" target="_blank" rel="noreferrer">https://git.reviewboard.kde.org/r/126856/diff/3#index_header</a><br />
 It was already reviewed by Milian Wolff. Feel free to pick anything useful from the old review diff into this one.</p></div>
</blockquote>

<p>Ah, that's a shame that review request had been ignored since. Seems the last version of that patch pretty much is covered by the patch here, just that this here drops the texts from the searchbar toolbuttons, given they have no use if an icon is set and a tooltip. So good to know we agree basically about what would be an improvement :)</p>

<p>The placeholder text might be worth to copy over, though Kate seems to have none (though Firefox has some). Undecided right now :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7061" rel="noreferrer">https://phabricator.kde.org/D7061</a></div></div><br /><div><strong>To: </strong>kossebau, KDevelop<br /><strong>Cc: </strong>igorkushnir, kdevelop-devel<br /></div>