<table><tr><td style="">valeriymalov 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/D3484">View Revision</a></tr></table><br /><div><div><p>I've tried to restore old behavior, so:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">If I run KWindowConfig::restoreWindowSize from dialog's constructor, it doesn't work on multi-monitor setup very well. windowHandle()→screen() seems to be set to leftmost screen (which isn't even primary in my case; is this a bug?), so restoreWindowSize loads saved size for same display no matter on which display the window is opened. Which makes resizing the dialog on any other screen has no effect on the next time it's opened.</li>
<li class="remarkup-list-item">If I run KWindowConfig::restoreWindowSize in a timer (like it used to be), it just messes up window positioning because the window is resized after it's been centered by KWin. Probably would mess up KWin rules too.</li>
</ul>
<p>Patch to demonstrate the issue: <a href="https://phabricator.kde.org/F5800781" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F5800781: restore_size.patch</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3484">https://phabricator.kde.org/D3484</a></div></div><br /><div><strong>To: </strong>valeriymalov, Plasma: Workspaces, aacid, mart<br /><strong>Cc: </strong>cfeck, davidedmundson, sebas, aacid, graesslin, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, apol, mart<br /></div>