Closed Bug 24474 Opened 25 years ago Closed 24 years ago

[EVENTTARG]ESPN page renders DOA on apprunner and viewer

Categories

(Core :: Layout: Tables, defect, P3)

Product:
Core
Component:
Layout: Tables
Platform:
x86
Other
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: mscott, Assigned: attinasi)

References

(
URL
)

Details

(Keywords: top100, Whiteboard: [PDT-])

Attachments

(2 files)

reduced case of mising panel
7.79 KB, text/html
Details
Reduced testcase for link rendered inactive by overlapping DIV
1.07 KB, text/html
Details 
Using 01/19 release build (I also tried this in viewer), when I visit the espn
website listed above, the page "kind of" renders and when we are done loading
the page, the page is dead. That is you can't click on any of the links or do
anything inside the page.

ESPN recently added some kind of complicated table widget on their homepage
where you can tab through different "pages" within this widget. I don't know how
else to describe it. This widget isn't being rendered correctly by layout today,
in fact it is coming up blank.

I'm choosing tables for the intial component assignment for this bug because I'm
wondering if there's a failure in parsing this table that is preventing the
document from properly being loaded. Would this cause all the links in the page
to be dead when you try to click on them though? I don't know....

Adding rickg to the cc list in case he can help me find the right owner for this.



Marking this as DOGFOOD because espn is one of the top 100 sites....
After writing this bug, I'm starting to think this probably isn't table related
at all. When I first load the page, there is a brief flicker where I can see the
contents inside of that complex table widget I described earlier. Then the
contents are blanked out and the page is dead. I wonder if it is a JS on load
problem...there is an awful lot of JS on their page.
I can't reproduce the problem on tonight's debug WinNT build. I also agree that
it doesn't sound like a table problem, but I'll let Rick reassign it.
Things got better in this morning's 01/20 build. The page doesn't come up dead.
i.e. i can click on the links.

However, the page is still not being rendered correctly. The table widget still
comes up blank and they have a pull down menu item which is showing up in the
wrong position on the screen.
Putting on PDT- radar.
Whiteboard: [PDT-]
FYI, there are several assigned bugs on espn.com, which address aspects
of the above description. However, the blank panel is new; and I see it 
(or rather don't see it) with 2000012008 win95 (I assume a problem with 
a DIVs hidden/visible).

Other www.espn.com (and its DNS aliases): 

 bug #14030 M16 <P></P> causes two line breaks on navigator, but only 1 on r
 bug #14033 M16 Extra whitespace between <FONT size=1> lines 	    	    
 bug #21560 M15 {TEXT] layout wrong, elements overlap 	   		    
 bug #21631 M15 [BLOCK] [NAV QUIRK] No vertical spacing between headlines in
 bug #22168 M14 [EVENTTARG] javascript: URL not working inside abs. pos. DIVs
 bug #22171 M15 [FLOAT] {css2} misplaced form widgets inside absolutely posi

of which (14033 & 21560) and (21631 & 14030) are essentially dups of each 
other.

Actually, the 'table widget' is primarily a collection of 'position: absolute'
DIVs, with simple tables and other elements contained within them.
Status: NEW → ASSIGNED
Target Milestone: M14
Troy, the test case involves absolutely positioned <div>s (there are no tables). 
Not all of them are showing up. Changing component to layout.
Assignee: karnaze → troy
Status: ASSIGNED → NEW
Component: HTMLTables → Layout
Chris, your test case looks just fine to me
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → WORKSFORME
QA Contact: chrisd → petersen
This bug may not belong to Troy. Nor may karnaze's test case apply. But the bug
I filed is still a problem and is not fixed. Re-opening.

I'm visiting www.espnet.sportszone.com using a 1/24 build on windows and this
table widget is still DOA. I am still not able to click on the tabs inside this
table widget and have it move between each tab envelope (is that the right
vocabulary for this widget?)

Let's work together to find the right owner for this if it isn't troy...
Status: RESOLVED → REOPENED
Chris, re-assigning back to you because the test case  you supplied works fine 
and so it sounds like the problem may be table related
Assignee: troy → karnaze
Status: REOPENED → NEW
Clearing WorksForMe resolution due to reopen.
Component: Layout → HTMLTables
QA Contact: petersen → chrisd
Resolution: WORKSFORME → ---
With a 1/26 (noon) debug WinNT build, I am seeing the content that was missing 
previously from the url and attachment. The links are messed up, however. In 
some cases they appear to be there but are not activated at the correct screen 
coordinates. In other cases they appear not to be there. Rod has agreed to take 
a look at this. Rod, you might first try throwing an anchor inside the 
attachment.
Assignee: karnaze → rods
I won't even look at this until the page renders correctly.
Have a look at bug #22168 which covers the problem of events not dispatched
to the tabs on the center panel of espn.go.com (and aliases). [This bug 
(#24474) is only unique for the 'missing content' in the center panel (actually 
a problem with hidden DIVs, but if they are working now then this is a dup).
Sorry, can't test myself today, as nsIFile landing broke win95B (what I run)].
As of M13 on Linux, the exact same problem occurs with www.lokigames.com.  (you

can see the screen for a second, and then only links).  Hope ya'll can track

this sucker down!


remove pdt- for dogfood.  add beta1 keyword to get on the beta1 review radar.
Keywords: beta1
Summary: [DOGFOOD] ESPN page renders DOA on apprunner and viewer → [DOGFOOD] top100- ESPN page renders DOA on apprunner and viewer
Whiteboard: [PDT-]
PDT+
Whiteboard: [PDT+]
adding top100 keyword.
Keywords: top100
Putting dogfood in the keyword field.
Keywords: dogfood
Summary: [DOGFOOD] top100- ESPN page renders DOA on apprunner and viewer → ESPN page renders DOA on apprunner and viewer
The fact that the links are not active is due to a problem of overlapping 
absolutely-positioned DIVs. Attached is a test case that simplifies the problem. 
Note that the position of the #TOPdataone and #TOPdatatwo DIVs are overlapping, 
and the part of the #TOPdatatwo DIV that 'blocks' the href in the #TOPdataone 
DIV makes that part of the link inactive. This really looks like a dup of bug 
12232... which I believe has a fix pending? But this one is PDT+ and 12232 is 
not.
Assignee: rods → attinasi
The problem of the links not working is a duplicate of either bug 12232 or bug
21034, which I both have fixed in my tree.  I'm working on (almost done, I
think) a rewrite of all the GetFrameForPoint methods to fix those bugs and a bug
that was never filed (but was mentioned in/part of bug 23161).

Right now, I've got lots of changes, and I'm not sure whether it's worth landing
all of them for M14, although it is probably an option (I think I could finish
the few last things I know need doing tonight - they're not that serious, and
I've been using the browser with my current changes).

The other option would be to try to isolate the part of the changes that fix
this bug, although I don't think that would be all that easy (it requires
changing lots of files.  There might be a trick with the view system that could
fix a certain very simple (and common) case, which may be this one, but I don't
know what it is.)

Does this bug cover only that the links don't work, or does it (still) cover
layout problems too?
Summary: ESPN page renders DOA on apprunner and viewer → [EVENTTARG]ESPN page renders DOA on apprunner and viewer
Removed PDT+ and Beta1 as recommended by Troy and Joki: the change has been made 
but is too risky to commit this close to beta.
Status: NEW → ASSIGNED
Keywords: beta1
Whiteboard: [PDT+]
Target Milestone: M14 → M15
Putting on PDT- radar for beta1.
Whiteboard: [PDT-]
This looks like it's working now, at least on Build ID 2000031708 on Linux.
Great job!

Unfortunately, the underlying event problem remains -- it's just that espn has
changed their page and they have stopped using the tabbed panels in the main 
page.
Fix checked in 2000-03-21 18:43 PST.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 24 years ago
Resolution: --- → FIXED
On Build ID 2000032305 on Linux, this still doesn't seem to work (today's page
at http://espn.go.com has the tabbed panels)  :-(

This is NOT fixed on the beta1 branch.  It is fixed on the trunk.  I assume a
build at 05 hours is a beta1 branch build...
Notwithstanding dbaron's correct note above (this fix is not in beta1 branch),
there is a second reason why the ESPN tabbed panels will not fully work with
dbaron's patch included: simply, the ESPN js is using 'document.layers' to 
detect Nav4, 'document.all' to detect IE, and so the js in the page will not
work in Mozilla. But that is an entirely different issue than the one that 
dbaron fixed (thanks for doing that!). 

Here is a testcase from back in December that is a better test (from bug 
#22168): 
   http://bugzilla.mozilla.org/showattachment.cgi?attach_id=3633
Verified fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: