Closed
Bug 161342
Opened 23 years ago
Closed 22 years ago
Help pages load as a modal window when launched from modal dialogs
Categories
(SeaMonkey :: Composer, defect)
Tracking
(Not tracked)
VERIFIED
DUPLICATE
of bug 195888
People
(Reporter: timeless, Assigned: oeschger)
Details
it shouldn't be modal.
Agreed. Ian, is this a dupe of bug 136647?
Assignee | ||
Comment 2•23 years ago
|
||
I don't know yet. I am looking into it, but right now I suspect these are two
different (if related) issues.
The file>publish dialog is modal. A window opened on top of it will itself be
modal, regardless of whether you asked it to be modal. See bug 99284 comment 3.
The "Sigh. I forget why we had to..." comment within is explained at least
partially by bug 15574; see especially bug 15574 comment 4.
Comment 5•23 years ago
|
||
So can we get around this by searching up the parent ".opener" chain until we
hit the main window and launch help from there? This, of course, should be done
centrally, not just for Composer's "Publish" dialog. Composer has lots of help
items for various dialog; I assume other apps/modules might also?
Summary: file>publish [help] loads help as a modal window → Help pages load as a modal window when launched from modal dialogs
Assignee | ||
Comment 6•23 years ago
|
||
I tested using window.opener in the openHelp()* function I define in
contextHelp.js some time ago, but with some other dialog, and got wonky behavior
by asking the actual opener's parent to open me instead. Weird modality. Since
we are trying to get all the Help script into one area anyway, shouldn't these
kinds of modality and parent choices be getting made in contextHelp.js--and then
made available in a general way?
* the preferred method for opening Help--to import this file and use openHelp()
http://lxr.mozilla.org/seamonkey/search?string=openHelp
comment 5: for notes on previous attempts to use the .opener chain, see bug 87720.
I'm not sure how much encouragement I want to lay out, but it may be possible to
fix this now. See bug 99284 comment 5. This bug is a duplicate of that one, by
the way. The code change is trivial but it wants a lot of testing.
*** This bug has been marked as a duplicate of 195888 ***
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → DUPLICATE
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•