cheerq / flexmdi

Automatically exported from code.google.com/p/flexmdi
0 stars 0 forks source link

MDIManager.manage() seems to be poorly documented #62

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
It isn't clear when you should use manage() on MDIManager, if at all.
manage() does nothing that add() doesn't do, and if you mistakenly call
both of them, manage() doesn't check to see if the window is already on the
window stack and just pushes it, which will cause problems when you close
the window due to having multiple references in the windowList.

Original issue reported on code.google.com by kldav...@gmail.com on 31 Mar 2008 at 3:47

GoogleCodeExporter commented 8 years ago
Thanks for the heads up, I thought we had changed it so that it would check the
existing windowList but I see that is not the case. manage() is intended to be 
used
for making an existing window part of the list of managed windows.

Also note that flexmdi has joined flexlib and issues should be logged there.
(http://code.google.com/p/flexlib/issues/list)

Thanks!
Ben

Original comment by ben.clin...@gmail.com on 31 Mar 2008 at 4:57