search

microscope-cockpit / cockpit

Cockpit is a microscope graphical user interface. It is a flexible and easy to extend platform aimed at life scientists using bespoke microscopes.
https://microscope-cockpit.org
GNU General Public License v3.0
35 stars 26 forks source link

bogus top level windows #617

Open carandraug opened 4 years ago

carandraug commented 4 years ago

On cockpit/gui/keyboard.py, when we iterate over the current top level windows, we skip over windows without a title because:

            # Sometimes we get bogus top-level windows; no idea why.
            # Just skip them.
            # \todo Figure out where these windows come from and either get
            # rid of them or fix them so they don't cause trouble here.

I was just doing a similar thing elsewhere and tracked down a bogus top-level window to the AuiManager in the LoggingWindow. I don't know why this is happening, it doesn't look like we are doing anything wrong with the AuiManager. I'll report it to wxWidgets.

This can be easily seen running wx.lib.inspection.InspectionTool().Show(). The bogus top level window will appear listed as "Frame ("frame")" under "Top-level Windows".

carandraug commented 4 years ago

Turns out that the issue is on wxWidgets itself not wxPython. I have reported it upstream https://trac.wxwidgets.org/ticket/18784

carandraug commented 3 months ago

Turns out that the issue is on wxWidgets itself not wxPython. I have reported it upstream https://trac.wxwidgets.org/ticket/18784

Upstream has move the bug tracker to github. It is now reported at https://github.com/wxWidgets/wxWidgets/issues/18784