<table><tr><td style="">kfunk accepted this revision.<br />kfunk added a comment.<br />This revision is now accepted and ready to land.
</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/D8872" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I like the idea.</p>

<p>The only problems I see is:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">This needs Qt 5.8 (should be ifdef'ed)</li>
<li class="remarkup-list-item">This call should probably be moved into <tt style="background: #ebebeb; font-size: 13px;">main()</tt>, even if that means linking <tt style="background: #ebebeb; font-size: 13px;">kdevelop</tt> against Qt5::Quick. This static function call affects the whole application and if we're doing this too late other components may initialize a quick window using the OpenGL backend already. (Think of KCMs being implemented in QtQuick, etc.)</li>
</ul>

<p>Opinions?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8872" rel="noreferrer">https://phabricator.kde.org/D8872</a></div></div><br /><div><strong>To: </strong>brauch, apol, mwolff, kfunk<br /><strong>Cc: </strong>kdevelop-devel<br /></div>