thomthom / SKUI

Ruby wrapper of classes that maps to GUI controls in SketchUp's UI::WebDialogs
MIT License
26 stars 14 forks source link

add_control() bug #60

Closed noelwarr closed 10 years ago

noelwarr commented 10 years ago

Say you have a groupbox with a button inside. You can add the group box to the window and then add the button to the group box. Or you can add the button to the group box and then the group box to the window. It doesn't matter.

Well it does if you're doing this dynamically to a window that is already visible and ready. This doesn't really break your UI, but it can be a bit confusing.

Checkout the example: https://gist.github.com/noelwarr/3677b35a9e377dad6628 This opens two windows. One will fail to show the button

thomthom commented 10 years ago

Good catch. Adding containers to open windows should inspect the child controls and add them to.