Closed
Bug 4279
Opened 26 years ago
Closed 25 years ago
[FEATURE] JavaScript Alert dialogs
Categories
(Core :: DOM: Core & HTML, defect, P3)
Core
DOM: Core & HTML
Tracking
()
VERIFIED
FIXED
M9
People
(Reporter: gerardok, Assigned: davidm)
References
Details
(Whiteboard: is this fixed now?)
Version: Seamonkey
Program: Viewer.exe
BuildID: mar_180610
Platform: WinNT
The JavaScript alert command is not opening any dialog box.
I've noticed it sends output to console. This failure possibly happens on other
platforms.
Steps to reproduce:
1. Create an HTML file with the following content:
<html>
<head>
<title>MyPage</title>
</head>
<body>
<script>
alert("Alert dialog opened");
</script>
</body>
</html>
2. Load that file in the browser
Actual Results: No dialog box is opened.
Expected Results: A dialog containing the "Alert dialog opened" string.
Updated•26 years ago
|
Status: NEW → ASSIGNED
Summary: JavaScript Alert dialogs not displayed
The same is happening on Linux and Mac, with the caviat that there is no
console to sent the output to on Mac. Platform changed to All.
Updated•26 years ago
|
Target Milestone: M6
Updated•26 years ago
|
Target Milestone: M6 → M7
The described failure applies to Apprunner also. Apprunner doesn't display any
output to the console. Severity changed to Major.
Updated•26 years ago
|
Assignee: vidur → don
Status: ASSIGNED → NEW
Target Milestone: M7
Comment 4•26 years ago
|
||
I just commited changes so that prompt() and confirm() are also defined (alert()
has been for a while). All of these calls get sent up to the implementor of
nsINetSupport. This is currently in browser land. The plumbing on the JS side
exists. It's up to one of Don's people to hook up the UI.
Assignee: don → davidm
Priority: P3 → P1
Summary: JavaScript Alert dialogs not displayed → [FEATURE] JavaScript Alert dialogs
Target Milestone: M8
The problem is that if I make the webshell window impliment the NetSuport
interface ( which I did in the past ), the dialogs don't load. I don't know why
but until necko lands I really don't want to look into it.
Comment 9•26 years ago
|
||
This is a stopper for people writing JavaScript and using alert() for debugging.
I suspect they can use dump() in the interim, but it would be great to at least
have alert() write out to the console as it does in Viewer.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•26 years ago
|
||
I checked in the code to hook up the UI. There are still problems in that you are
unable to bring up the dialog from certain points in the code but I believe I
have bugs on most of them
Reporter | ||
Comment 11•26 years ago
|
||
Re-tested on 07/28 build. A dialog box opens now but the alert msg is not being
displayed. A dialog box containing the "Alert dialog opened" string is expected.
Bug reopened.
Reporter | ||
Comment 12•26 years ago
|
||
Re-tested on 07/28 build. A dialog box opens now but the alert msg is not being
displayed. A dialog box containing the "Alert dialog opened" string is expected.
Bug reopened.
Assignee | ||
Comment 13•26 years ago
|
||
*** Bug 5127 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 14•26 years ago
|
||
The dialogs should come up. However they will not be modal until after danm
fixes 11611 so Confirm and Prompt will not return correct values.
Updated•26 years ago
|
Whiteboard: is this fixed now?
Status: NEW → RESOLVED
Closed: 26 years ago → 26 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 15•26 years ago
|
||
fix checked in.
Reporter | ||
Comment 16•26 years ago
|
||
Verified fix on 8/20 build on Win NT and MacOS, and 8/24 on Linux.
Comment 17•25 years ago
|
||
This bug is still marked as a JavaScript issue at:
http://www.mozilla.org/projects/seamonkey/release-notes/m12-detail.html#general_browser
Re-opening to get this oversight covered.
Severity: major → normal
Status: VERIFIED → REOPENED
Priority: P1 → P3
Resolution: FIXED → ---
Comment 18•25 years ago
|
||
NOTE: Also downgraded to reduce radar detection.
Assignee | ||
Comment 19•25 years ago
|
||
closing. Open a bug against the release note people.
Status: REOPENED → RESOLVED
Closed: 26 years ago → 25 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•