Closed
Bug 84209
Opened 23 years ago
Closed 12 years ago
window onclose handlers need love
Categories
(SeaMonkey :: MailNews: Message Display, defect)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: danm.moz, Unassigned)
Details
(Whiteboard: [waiting for comment 2 answer] [patchlove] [2012 Fall Equinox])
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
As we were discussing, window onload handlers should return a value: if true,
the window is closed automatically; if false it isn't (I think I got that
right). Several onclose handlers in Mozilla are returning unspecified values,
and several are closing the window themselves (which is guarded against, so it
doesn't hurt, but it is unnecessary and not -- strictly speaking -- correct.)
I'll attach a patch that I was working on recently when I thought I was going to
have to fix this. I'm not saying it's all correct, but it addresses all the
problems I could find. Maybe you want to be the first owner, and hand it along,
or maybe, heck, you just want to hand this bug back to me.
Comment 2•23 years ago
|
||
Dan, may I ask what this helps? Does it fix anything in particular, or is it
just stupid me? :)
QA Contact: esther → stephend
Updated•20 years ago
|
Product: Browser → Seamonkey
Updated•20 years ago
|
Assignee: sspitzer → mail
Updated•16 years ago
|
Assignee: mail → nobody
QA Contact: stephend → message-display
Comment 3•16 years ago
|
||
This bug is being marked EXPIRED as it has seen no activity in a very long time.
If you think that the issue reported might still be relevant, please test with a recent release of SeaMonkey and if the problem persists feel free to re-open the report. Thank you.
http://www.seamonkey-project.org/
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → EXPIRED
Comment 4•16 years ago
|
||
Bulk reopening incorrectly expired bugs - no activity does not constitute no bug - these need proper checking.
Status: RESOLVED → REOPENED
Resolution: EXPIRED → ---
Updated•15 years ago
|
Status: REOPENED → NEW
Whiteboard: [patchlove]
Updated•15 years ago
|
Whiteboard: [patchlove] → [waiting for comment 2 answer] [patchlove]
Comment 5•15 years ago
|
||
Sadly, I cannot provide any additional insight here, because I'm not the "Dan M" in question -- that's Dan Matejka.
Comment 6•12 years ago
|
||
This bug is being closed INCOMPLETE for lack of an answer to comment #2 more than 3 years after it was requested from the reporter in the Whiteboard.
Feel free to REOPEN with a comment providing the requested information.
Status: NEW → RESOLVED
Closed: 16 years ago → 12 years ago
Resolution: --- → INCOMPLETE
Whiteboard: [waiting for comment 2 answer] [patchlove] → [waiting for comment 2 answer] [patchlove] [2012 Fall Equinox]
You need to log in
before you can comment on or make changes to this bug.
Description
•