<table><tr><td style="">zzag 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/D26233">View Revision</a></tr></table><br /><div><div><p>Urgh, the xdg-shell spec is ambiguous about the effective window geometry. Some compositors clamp the window geometry at commit time, some compositors clamp at query time (like this patch does), and some don't clamp at all. :/</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>For example when attached buffer size changes.</p></blockquote>

<p>We can't connect to <tt style="background: #ebebeb; font-size: 13px;">SurfaceInterface::sizeChanged</tt> since this signal is emitted only for the main surface if I recall correctly. I guess we could compute the bounding rect in the commit handler.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26233">https://phabricator.kde.org/D26233</a></div></div><br /><div><strong>To: </strong>davidedmundson, KWin, zzag<br /><strong>Cc: </strong>romangg, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>