theodox / mGui

Python module for cleaner maya GUI layout syntax
MIT License
123 stars 23 forks source link

Heads up on popupmenus #87

Closed theodox closed 6 years ago

theodox commented 7 years ago

I noticed and fixed an issue with mGui 2.2 and popup menus . The new and improved menu syntax had the side effect of making popups not get added to their owning widgets -- which made them get garbage collected, so any references to themselves (like kwargs['sender']) would result in dead reference errors.

I've fixed it already in e8378c0 but if you run into that you may need to catch up on that commit

theodox commented 7 years ago

I'll close the issue with the next point release

bob-white commented 7 years ago

I had to tweak the shelf_loader a bit with this change, but it looks like that was the only place that relied on the previous behavior.

theodox commented 7 years ago

One more change: I fixed a bug in the menu loader that didn't properly return the parentage of submenus. Should be good. Think we're on track for 2.2.1?

theodox commented 6 years ago

I'm going to close this issue, I think it's been stable for a while