<table><tr><td style="">kmaterka created this revision.<br />kmaterka added reviewers: davidedmundson, Plasma, Plasma: Workspaces.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />kmaterka requested review of this revision.
</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/D24529">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>If needed, perform resize in each update().<br />
Client window may be too big, it must be resized to a certain reasonable size. Sometimes resizing in the constructor is not enough because it is too early and the client window is not fully initialized. Even if client window is resized, on first update it reverts back to original size (or just ignores the first request - some kind of a race condition).</p>

<p>In addition, I applied some rules from: <a href="https://community.kde.org/Policies/Kdelibs_Coding_Style" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Policies/Kdelibs_Coding_Style</a></p>

<p>BUG: 358240</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I tested it using keepass2, now it shows better results (still not perfect). Window is resized to 32x32, but icon is only 24x24 it top-left. Better than 273x273 with tiny dot in the corner. This is a workaround for unsolvable bug in Mono - AFAIK Windows Forms are not longer maintained so it won't be fixed.</p>

<p>I tested this change with several other apps for regressions, no issues:<br />
pidgin<br />
liferea<br />
hexchat<br />
tuxguitar<br />
xchat<br />
keepassx<br />
stardict<br />
and other I don't remember now. No regressions.</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/D24529">https://phabricator.kde.org/D24529</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>xembed-sni-proxy/main.cpp<br />
xembed-sni-proxy/sniproxy.cpp<br />
xembed-sni-proxy/sniproxy.h</div></div></div><br /><div><strong>To: </strong>kmaterka, davidedmundson, Plasma, Plasma: Workspaces<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>