Closed
Bug 41301
Opened 25 years ago
Closed 22 years ago
In Editor, IFRAME is editable instead of page (editor not created)
Categories
(Core :: DOM: Editor, defect, P3)
Core
DOM: Editor
Tracking
()
People
(Reporter: rubydoo123, Assigned: Brade)
References
()
Details
(Whiteboard: editorbase-)
Attachments
(1 file)
(deleted),
text/html
|
Details |
What happens is the file never completes loading in the editor. All the content
is there, but the throbber is still spinning. This means there is no
nsIHTMLEditor created for the document. This is probably why the NS_NO_INTERFACE
error keeps getting thrown from nsEditorShell. After more digging, if the IFRAME
is commented out, then the page loads and is editable.
Code snippet to use for testing:
<html>
<head><title>test</title></head>
<body>
<p>Here is an iframe:</p>
<iframe frameborder=1 width=20 height=20>IFRAME element</iframe><br>
</body>
</html>
I believe iframes are out... but I may be wrong. Nisheeth are these handled in
the DOM?
Assignee: gagan → nisheeth
Comment 2•24 years ago
|
||
The HTML spec says (sec. 16.5):
The information to be inserted inline is designated by the src attribute of
this element. The contents of the IFRAME element, on the other hand, should
only be displayed by user agents that do not support frames or are configured
not to display frames.
IFRAMEs currently work fine, but this bug exists whether the IFRAME has a SRC
attribute or not. However, the *contents* of the IFRAME do become editable.
Attaching a testcase.
TO REPRODUCE:
* load attached testcase
* Go to "File | Edit Page"
ACTUAL RESULTS:
* The contents of the iframe are editable, the page is not.
EXPECTED RESULTS:
* The page should be editable. The contents of the iframe should not (?).
This seems more like an editor bug than a Layout bug. (Note that I never saw
the throbber keep spinning. Maybe that is fixed.)
Summary: Web page containing IFRAME, never loads → In Editor, IFRAME is editable instead of page
Comment 3•24 years ago
|
||
Comment 4•24 years ago
|
||
Beth, the page loads fine now, but as David says, instead of the page becoming
editable, the IFRAME inside the page becomes editable. This seems like a editor
specific issue, so re-assigning this back to you.
Assignee: nisheeth → beppe
Component: Networking → Editor
Reporter | ||
Comment 5•24 years ago
|
||
asking Simon if he has any ideas as to why the page becomes uneditable, but the
IFRAME is editable.
Assignee: beppe → sfraser
Comment 6•24 years ago
|
||
Yes, I do. We'll make the editor on the last IFRAME that is loaded, blowing away
previous editors in the process. Fixing this requires some thought.
Testcase, please?
Status: NEW → ASSIGNED
Comment 7•24 years ago
|
||
Actually, I think fixing this will requires at least some of the work for
embedding to be complete (bug 35639).
Reporter | ||
Comment 8•24 years ago
|
||
Simon -- tests cases are already in here, see the sample code in the file and
David Baron attached a test case
Comment 10•24 years ago
|
||
cc: valeski, dougt for embedding info. To fix this bug, I need to be able to hang
an editor off a docShell (or equivalent), which leads us down the embedding path.
Have you guys started down that path yet?
Depends on: 34477
Target Milestone: M18 → M20
Reporter | ||
Comment 11•24 years ago
|
||
added correctness keyword and bringing it back to m19
Keywords: correctness
Target Milestone: M20 → M19
Reporter | ||
Comment 12•24 years ago
|
||
moving to future per review with bij and beppe
Target Milestone: M19 → Future
Comment 13•24 years ago
|
||
*** Bug 60869 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 15•22 years ago
|
||
In a recent build I now see this:
page loads fine but it not editable; clicking on links in the iframe load those
pages
Assignee | ||
Comment 16•22 years ago
|
||
nsbeta1+
editorbase- for now but this is subject to change later
Assignee | ||
Comment 17•22 years ago
|
||
add http://news.bbc.co.uk/ as a testcase (iframe with java)
Assignee | ||
Comment 18•22 years ago
|
||
duping
*** This bug has been marked as a duplicate of 14156 ***
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•