<table><tr><td style="">croick created this revision.<br />Restricted Application added a subscriber: kdevelop-devel.
</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/D6542" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Fixes to GrepOutputView toolbar</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">fix crash when aborting grep jobs at startup by clicking on "Clear Search History" (introduced in commit <a href="https://phabricator.kde.org/R33:42eb6bb62c28923e5f6f4ff0cee3e8e6ee07bd2e" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">42eb6bb62c28</a>)</li>
<li class="remarkup-list-item">correctly disable result navigation buttons (fixes crash when clicking on "Collapse All" button for an empty result)</li>
<li class="remarkup-list-item">change icons for "Collapse All" and "Expand All" (were marked as TODO)</li>
</ul>

<p>Before:<br />
<a href="https://phabricator.kde.org/F3805931" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F3805931: old.png</a></p>

<p>After:<br />
<a href="https://phabricator.kde.org/F3805930" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F3805930: new.png</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">quit KDevelop with a long GrepJob in history</li>
<li class="remarkup-list-item">restart KDevelop and abort job by clicking on "Clear Search History" -> no crash</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">start a long GrepJob and stop it by clicking on "Clear Search History"</li>
<li class="remarkup-list-item">navigation buttons are now disabled</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>BRANCH</strong><div><div>grep</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6542" rel="noreferrer">https://phabricator.kde.org/D6542</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/grepview/grepdialog.cpp<br />
plugins/grepview/grepdialog.h<br />
plugins/grepview/grepoutputview.cpp<br />
plugins/grepview/grepoutputview.h<br />
plugins/grepview/grepviewplugin.cpp<br />
plugins/grepview/grepviewplugin.h</div></div></div><br /><div><strong>To: </strong>croick, KDevelop<br /><strong>Cc: </strong>kdevelop-devel<br /></div>