<table><tr><td style="">elvisangelaccio created this revision.<br />elvisangelaccio added a reviewer: rthomsen.<br />elvisangelaccio added a subscriber: kde-utils-devel.<br />elvisangelaccio set the repository for this revision to rARK Ark.<br />elvisangelaccio added a project: Ark.</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/D1829" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p><tt style="background: #ebebeb; font-size: 13px;">MainWindow </tt>inherits from <tt style="background: #ebebeb; font-size: 13px;">KXmlGuiWindow</tt>, but we never call <tt style="background: #ebebeb; font-size: 13px;">KXmlGuiWindow::setupGUI()</tt>. This is bad because we lose a lot of things (for example the "Show Toolbar" checkbox in the Settings menu) that we have to manually provide (e.g. the configure shortcuts/toolbar dialogs).</p>

<p>Calling <tt style="background: #ebebeb; font-size: 13px;">setupGUI()</tt> also ensures that the window size be automatically restored (<tt style="background: #ebebeb; font-size: 13px;">setAutoSaveSettings()</tt> is implicitly called).<br />
The call to <tt style="background: #ebebeb; font-size: 13px;">resize()</tt> is also useless (640x480 seems to be already the default) and possibly bad, according to: <a href="https://api.kde.org/frameworks/kxmlgui/html/classKMainWindow.html#a421a8ca496cd11ff388b630357735cf4" class="remarkup-link" target="_blank" rel="noreferrer">https://api.kde.org/frameworks/kxmlgui/html/classKMainWindow.html#a421a8ca496cd11ff388b630357735cf4</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Check whether is still possible to edit shortcuts and the toolbar.<br />
Check whether the new "Show Toolbar" checkbox works.</p>

<p>This also fixes bug 357660: <a href="https://bugs.kde.org/show_bug.cgi?id=357660" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=357660</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rARK Ark</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1829" rel="noreferrer">https://phabricator.kde.org/D1829</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>app/mainwindow.cpp<br />
app/mainwindow.h</div></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>elvisangelaccio, rthomsen<br /><strong>Cc: </strong>kde-utils-devel, tctara<br /></div>