<table><tr><td style="">sharvey 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/D12252">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Once that's fixed, there's still one more issue here: always calling <tt style="background: #ebebeb; font-size: 13px;">resize()</tt> this means that once the user resizes the window, that custom size will be overridden on the next launch. We want to set a better default size, not enforce a mandatory size. I wonder if we should just remove the <tt style="background: #ebebeb; font-size: 13px;">resize()</tt> call and change <tt style="background: #ebebeb; font-size: 13px;">setMinimumSize()</tt> to enforce 1024x700? Most KCMs look pretty bad at smaller sizes anyway.</p></blockquote>

<p>I thought about that. I figured there wasn't any harm in letting the user resize the window even if it wouldn't be saved. Do we want to go with the standard routine of saving the window geometry and restoring it the next time the user launches the application? I can work that in, maybe with a <tt style="background: #ebebeb; font-size: 13px;">firstLaunch</tt> boolean flag that only triggers the manual setting of the window size on the very first run. The window still won't get smaller than <tt style="background: #ebebeb; font-size: 13px;">minimumSize</tt>, but if someone wants to enlarge it, we could save that size and restore it in the future.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R124 System Settings</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12252">https://phabricator.kde.org/D12252</a></div></div><br /><div><strong>To: </strong>sharvey, ngraham, mart<br /><strong>Cc: </strong>zzag, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>