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)

x86
Windows 2000
defect
Not set
normal

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?
I don't know yet. I am looking into it, but right now I suspect these are two different (if related) issues.
--->Ian
Assignee: syd → oeschger
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.
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
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
v
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.