https://bugzilla.wikimedia.org/show_bug.cgi?id=69630

Roan Kattouw <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #4 from Roan Kattouw <[email protected]> ---
(In reply to Juliusz Gonera from comment #3)
> I thought it could be easily fixed by adding this.inspectors.clearWindows()
I haven't reported this in Bugzilla yet, but I discovered yesterday that
clearWindows() is completely broken. It will only clean up the current window
(i.e. the one that's currently open), and it won't touch any others.

> in ve.ui.Context.prototype.destroy, but unfortunately that results in:
> 
> "Uncaught TypeError: Cannot read property 'getComputedStyle' of undefined "
> 
> This happens when WindowManager invokes OO.ui.Window.prototype.hold. I
> suspect it's because everything in WindowManager is async/delayed (to
> accommodate for animations), so DOM nodes no longer exist when WindowManager
> tries to tear windows down.
Yeah that sounds likely. Maybe WindowManager needs a destroy function that
destroys everything immediately? I also don't really understand why
WindowManager not cleaning up is causing problems if the WindowManager's DOM
element is removed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to