yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix #39172. Send the synthetic configure notify event _after_ reparenting

Browse source 
into WindowWrapper (in WindowWrapper::show() triggered by Client::show()).
Some toolkits go up the window hiearchy when finding the "real" position
of the window (i.e. the topleft decoration border corner), and if they
receive the ReparentNotify after the synthetic ConfigureNotify, they'll
miscalculate it. (Hmm, I wonder how Qt does it since it gets it right.)

CCMAIL: 39172-done@bugs.kde.org

svn path=/trunk/kdebase/kwin/; revision=185453
This commit is contained in:
Luboš Luňák 2002-10-24 15:47:09 +00:00
parent 4304971dfd
commit 7c33e280ef
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
client.cpp
View file

@ -874,7 +874,6 @@ bool Client::manage( bool isMapped, bool doNotShow, bool isInitial )
bool showMe = (state == NormalState) && isOnDesktop( workspace()->currentDesktop() ); bool showMe = (state == NormalState) && isOnDesktop( workspace()->currentDesktop() );
sendSyntheticConfigureNotify();
workspace()->clientReady( this ); // will call Workspace::propagateClients() workspace()->clientReady( this ); // will call Workspace::propagateClients()
if ( showMe && !doNotShow ) { if ( showMe && !doNotShow ) {
@ -914,6 +913,8 @@ bool Client::manage( bool isMapped, bool doNotShow, bool isInitial )
if ( !doNotShow ) if ( !doNotShow )
workspace()->updateClientArea(); workspace()->updateClientArea();
sendSyntheticConfigureNotify();
delete session; delete session;
return showMe; return showMe;
} }