<table><tr><td style="">brauch 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/D3110" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Good change in general, but why don't you simply start the timer at the place where showProgress() was emitted before, in case it's not running yet? Then you wouldn't need the "toggled on/off" state for the timer.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKDEVPLATFORM KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3110" rel="noreferrer">https://phabricator.kde.org/D3110</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>wcpan, kfunk, KDevelop<br /><strong>Cc: </strong>brauch, kfunk, kdevelop-devel<br /></div>