The debugger needs a way to temporarily disable the modality of a modal window for bug 100677. Dan and I discussed this and he says he could add an |isModal| property to nsIXULWindow to take care of this. 0.9.9, right?
Created attachment 67323 [details] [diff] [review] switch on and off a modal window's modal UI behaviour Robert -- try this patch; let me know if it does what you need. You'll probably hate the name I chose for the attribute. Heh. I wanted something that didn't collide with other terms in use or imply too much. Obscurity was called for. Embrace your appearance; learn to live within your miens.
How about we call the attribute |modalificated|? I really thought I was on to something with the bug summary (if you excuse the missing preposition.) I'll give this a shot tonight and report on my findings.
Created attachment 67806 [details] [diff] [review] venkman patch -- not for checkin, yet. modalMien works for me. This patch, which depends on bug 121039, actually uses it. To test, start venkman and enter "fbreak commonDialog.js 200", then "evald alert(1)", and click "ok" on the alert window. Without Dan's patch the app locks up as the script context of the modal window is disabled while it is still modal. With the patch everything goes much smoother.
Comment on attachment 67323 [details] [diff] [review] switch on and off a modal window's modal UI behaviour r=rginda
Attachment #67323 - Flags: review+
I believe that this bug and patch (the first patch, anyway) are no longer in fashion, to be replaced by the alternate implementation covered by bug 126786. 0.99 cleanup time: I'm closing this bug in favour of its better half. Reopen, of course, if I'm mistaken.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.