Closed
Bug 24474
Opened 25 years ago
Closed 25 years ago
[EVENTTARG]ESPN page renders DOA on apprunner and viewer
Categories
(Core :: Layout: Tables, defect, P3)
Tracking
()
VERIFIED
FIXED
M15
People
(Reporter: mscott, Assigned: attinasi)
References
()
Details
(Keywords: top100, Whiteboard: [PDT-])
Attachments
(2 files)
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....
Reporter | ||
Comment 1•25 years ago
|
||
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.
Comment 2•25 years ago
|
||
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.
Reporter | ||
Comment 3•25 years ago
|
||
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.
Comment 5•25 years ago
|
||
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.
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M14
Comment 6•25 years ago
|
||
Comment 7•25 years ago
|
||
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
Updated•25 years ago
|
QA Contact: chrisd → petersen
Reporter | ||
Comment 9•25 years ago
|
||
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
Comment 10•25 years ago
|
||
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
Comment 11•25 years ago
|
||
Clearing WorksForMe resolution due to reopen.
Component: Layout → HTMLTables
QA Contact: petersen → chrisd
Resolution: WORKSFORME → ---
Comment 12•25 years ago
|
||
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
Comment 13•25 years ago
|
||
I won't even look at this until the page renders correctly.
Comment 14•25 years ago
|
||
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)].
Comment 15•25 years ago
|
||
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!
Comment 16•25 years ago
|
||
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-]
Updated•25 years ago
|
Summary: [DOGFOOD] top100- ESPN page renders DOA on apprunner and viewer → ESPN page renders DOA on apprunner and viewer
Assignee | ||
Comment 20•25 years ago
|
||
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
Assignee | ||
Comment 21•25 years ago
|
||
Comment 22•25 years ago
|
||
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
Assignee | ||
Comment 23•25 years ago
|
||
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.
Comment 25•25 years ago
|
||
This looks like it's working now, at least on Build ID 2000031708 on Linux.
Great job!
Comment 26•25 years ago
|
||
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.
Comment 27•25 years ago
|
||
Fix checked in 2000-03-21 18:43 PST.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
Comment 28•25 years ago
|
||
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) :-(
Comment 29•25 years ago
|
||
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...
Comment 30•25 years ago
|
||
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
You need to log in
before you can comment on or make changes to this bug.
Description
•