<table><tr><td style="">kossebau added inline comments.
</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/D4287" rel="noreferrer">View Revision</a></tr></table><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/D4287#inline-17127" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">arrowdodger</span> wrote in <span style="color: #4b4d51; font-weight: bold;">templateclassassistant.cpp:519</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Can't we just do <tt style="background: #ebebeb; font-size: 13px;">currentPage()->setFocusToFirstEditWidget()</tt> after all those <tt style="background: #ebebeb; font-size: 13px;">if</tt>s?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Good point, I forgot to ask some questions I have about this:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">The repeated explicit calls could be avoided by making <tt style="background: #ebebeb; font-size: 13px;">setFocusToFirstEditWidget()</tt> a part of the interface. What would be the best (KDevelop) style here: a) add an abstract interface which all pages would optionally implement and which would be queried here first or b) add another subclass with that abstract method added which all pages need to subclass from?</li>
<li class="remarkup-list-item">Would it make more sense to set the first edit widget as focus proxy to the page container? So there is no need for a special call with a given semantic <tt style="background: #ebebeb; font-size: 13px;">setFocusToFirstEditWidget()</tt> here we just need to call setFocus() on the page? That approach would also mean any click in the page background would set the keyboard focus to the first edit widget, is that expected?</li>
</ol></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4287" rel="noreferrer">https://phabricator.kde.org/D4287</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>kossebau, KDevelop<br /><strong>Cc: </strong>arrowdodger, kdevelop-devel<br /></div>