5418 - some unspecified JS errors

Status: VERIFIED FIXED

(Core :: DOM: Core & HTML, defect, P3)

Description

[html-bin]

Go to the above URL in mozilla M4 and see for yourself, visit in any other
browser and this will not happen, also some of the JS doesnt work but I guess
thats really minor at this point of development see the source of that page for
more info

Comment 1

[karnaze (gone)]

Moving to M8.

Comment 2

[John Morrison]

Attachment: testcase; <font><img><br><img></font> shows Nav4 vs CSS2 line box bug

Comment 3

[John Morrison]

The test case (http://bugzilla.mozilla.org/showattachment.cgi?attach_id=396)
shows another manifestation of Nav4 vs CSS2 line box incompatibility, i.e.,
bug #5821. (The original page showed the typical <a><img></a> case, but this
<font><img><br><img></font> variation was also present. (The gaps have nothing
to do specifically with the HTMLTables).

 Ian (py8ieh): do you want to take this on Kipp's behalf (either as a
duplicate of #5821 or as a separate example of the same problem).

Comment 4

[Hixie (not reading bugmail)]

What about the JS problems?

Comment 5

[John Morrison]

Ah. Good point. (Sorry, I did gloss over those; looks again ...)

 The principal js error on the page is that window.confirm is not
defined.  However, that is bug #5127 (and on bug #4279, vidur says that he
checked this code in yesterday).

 [Otherwise, the rollovers are sluggish, and on occasion image loading
fails but I don't believe those are unique to this page (and are more
netlib->necko than js I believe) --- in other words, I'm not inclined to
track these perf issues through this bug].

[e-mail] : were there other js errors for this page, that I am
missing?

Comment 6

[Hixie (not reading bugmail)]

[Removing myself from cc list, updating summary, updating component. Since the
line box 'bug' has already plenty of examples, I suggest this bug concentrate
on the javascript issues.]

Comment 7

[leger]

Updating component.  Please do not use Javascript.  Use Javascript Engine.
Please always check component list at:
http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 8

[karnaze (gone)]

Moving to M9.

Comment 9

[karnaze (gone)]

The attachment doesn't have a table in it. Its images wouldn't load for some
reason so I substituted them with iefavorite.gif. Reassigning to Kipp.

Comment 10

[kipp]

The layout and JS issues have been resolved for this page. There was a small
error with handling "window.confirm" that was a side effect of the necko landing
which I fixed with vidur's help. The layout issues were already resolved in
5821.

Comment 11

[cbegle]

since kipp says it's a window.confirm thing, i've updated the component and qa
contact.

Comment 12

[Prashant Desale]

Verified.