<table><tr><td style="">sars 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><p>Sorry, I was a bit busy when this review came and then I forgot about it :(</p>

<p>Good:<br />
Show ToolView when build is invoked but is already in progress<br />
Rename displayBuildResult->postMessage to fit new use cases<br />
Use postMessage indstead of KMessageBox on some infos because it is less annoying</p>

<p>I'm not sure about the auto-hiding. Quite often I want to see the result after the build.</p>

<p>Thanks for pushing improvements! :)</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D16336#inline-95841">View Inline</a><span style="color: #4b4d51; font-weight: bold;">plugin_katebuild.cpp:740</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">QRegExp</span> <span class="n">rx</span> <span style="color: #aa2211">=</span> <span class="n">QRegExp</span><span class="p">(</span><span class="n">QStringLiteral</span><span class="p">(</span><span style="color: #766510">"^</span><span style="color: #bb6622">\\</span><span style="color: #766510">[([</span><span style="color: #bb6622">\\</span><span style="color: #766510">s</span><span style="color: #bb6622">\\</span><span style="color: #766510">d</span><span style="color: #bb6622">\\</span><span style="color: #766510">%]+)</span><span style="color: #bb6622">\\</span><span style="color: #766510">]"</span><span class="p">));</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span style="color: #aa4000">if</span> <span class="p">(</span><span class="n">line</span><span class="p">.</span><span class="n">contains</span><span class="p">(</span><span class="n">rx</span><span class="p">))</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">            <span style="color: #74777d">// QString name = fetchPluginName();</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The idea is good, but until it works it is best to not output debug messages.</p>

<p style="padding: 0; margin: 8px;">If it would be possible to use the same plasma integrated progress indication as dolphin it would be nice, but if it would be only in the toolview you kinda already have it there.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></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>To: </strong>loh.tar, Kate, sars<br /><strong>Cc: </strong>cullmann, sars, kwrite-devel, Kate, hase, michaelh, ngraham, demsking, dhaumann<br /></div>