IFRAMES and JavaScript

VERIFIED FIXED in M9

Status

()

P3
normal
VERIFIED FIXED
20 years ago
20 years ago

People

(Reporter: bmurtha, Assigned: harishd)

Tracking

Trunk
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

20 years ago
Hello,

   I'm really bummed, I used to work for LookSmart and we had a hellva' time trying to find a way to use javascript and iframes so that we could display the same page dynamic content to the client. This took months of debugging and coding, and when I opened and saw it broke again, I almost cried. Is their anyway you guys could engineer some backwards compatiblity. I mean, I think it's great that you guys are supporting I frames. They will make like much easier for everyone. And maybe for this type of work we should have used a browser detect. I just wanted you guys to know my pain. Which really don't matter, I don't work for that company anymore and so it's someone else's problem. But I did just want to share this with you.

P.S.

 Aside from this, you guys are kickin' ass with this thing. I was a Netscape person, gone IE, but seriously thinking of changing camps again. It all boils down to who makes a better browser. And from what I can see, you guys are winning.

Regards,

Bryan

Updated

20 years ago
Assignee: leger → rickg
Component: JavaScript → Layout
QA Contact: leger → petersen

Comment 1

20 years ago
Updating QA contact and component.  (Please do not use Javacript.  Let's try
Layout guys, and if this is indeed a Javascipt issue, assigne to Javascript
Engine)

Updated

20 years ago
Assignee: rickg → vidur

Comment 2

20 years ago
Vidur -- can you make sense of this?

Updated

20 years ago
Assignee: vidur → harishd

Comment 3

20 years ago
The main layout issue on this page is that the block contents of the <IFRAME>s
are being pushed out of the IFRAME. In reality, the should be children of the
IFRAME (and the layout code doesn't render them).

I'm attaching a reduced test case and a proposed patch. Note that this should be
reasonable similar to what we do for APPLETs, EMBEDs and OBJECTs, right?

There is another problem with the ILAYER/IFRAME below the hrule. The source of
the IFRAME is a https: URL that we can't seem to load (Note that Navigator 4.0
can't seem to load it either, though it loads the ILAYER URL without a problem.
Is the server doing some sort of sniffing?).

Comment 4

20 years ago
Oops. Navigator did load the IFRAME URL - it just took a long time.

Comment 5

20 years ago
Created attachment 441 [details]
reduced test case for content inside IFRAME

Comment 6

20 years ago
Created attachment 442 [details] [diff] [review]
Proposed patch
(Assignee)

Updated

20 years ago
Target Milestone: M9
(Assignee)

Updated

20 years ago
Status: NEW → RESOLVED
Last Resolved: 20 years ago
Resolution: --- → FIXED
(Assignee)

Comment 7

20 years ago
Per spec. IFRAME can contain "flow" elements.  Thanx vidur.
The fix is in.

Marking bug fixed.

Updated

20 years ago
Status: RESOLVED → VERIFIED

Comment 8

20 years ago
Fixed in the July 16th Build (All platforms: Mac, Windows, and Linux)
You need to log in before you can comment on or make changes to this bug.