<table><tr><td style="">loh.tar updated this revision to Diff 47717.<br />loh.tar edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-lnypj6bfqwoxixd/">(Show Details)</a><br />loh.tar edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-giirpcmqu5dlzy7/">(Show Details)</a><br />loh.tar 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/D16336">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Fix switch/case coding style</li>
<li class="remarkup-list-item">QRegExp -> QRegularExpression</li>
<li class="remarkup-list-item">Remove 'Build still in progress' hint because now is always such hint active and besides, it would be immediately replaced again</li>
<li class="remarkup-list-item">Only auto-hide on default target. I think this could be acceptable now</li>
<li class="remarkup-list-item">Add progressinfo as KTextEditor::Message</li>
</ul>

<p>Hint</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">buildCurrentTarget() returns bool but is never used</li>
</ul>

<p>Possible change: Only show progress hint when view is not visible</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span>- Show ToolView when build is invoked but is already in progress<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span>- Auto hide ToolView on successful <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">build of</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">'default</span> target<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> 'B</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">' b</span>uild<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">'</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span>- Rename displayBuildResult->postMessage to fit new use cases<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span>- Use postMessage indstead of KMessageBox on some infos<br />
   <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">   </span>because it is less annoying<br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Add progressinfo as KTextEditor::Message</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- </span>Invoke build by some short cut key<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
- Invoke build by buttons<br />
- Cancel running build and invoke build again<br />
- Force error/warning when compile<br />
- Force wrong build commands<br />
</span></div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D16336?vs=47571&id=47717">https://phabricator.kde.org/D16336?vs=47571&id=47717</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16336">https://phabricator.kde.org/D16336</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>addons/katebuild-plugin/plugin_katebuild.cpp<br />
addons/katebuild-plugin/plugin_katebuild.h</div></div></div><br /><div><strong>To: </strong>loh.tar, Kate, sars<br /><strong>Cc: </strong>yurchor, cullmann, sars, kwrite-devel, Kate, hase, michaelh, ngraham, demsking, dhaumann<br /></div>