Closed Bug 2392 Opened 26 years ago Closed

[PP]CSS support not working at HotBot site

Categories

(Core :: DOM: HTML Parser, defect, P2)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: chrispetersen, Assigned: rickg)

References

()

Details

Version: Viewer.exe Build: Thurs Jan 14th Platform: Windows NT Other Platforms: Windows 98, Win 95. Expected Results: The HotBot site to be use CSS to layout images correctly. What I got: The HotBot logo and text are displayed flush left and appear on top of each other. Steps to Reproduce: 1) Go to www.hotbot.com 2) Notice the way the layout is displayed. The Hotbot logo and text are placed flush left.
Assignee: peterl → troy
Looks like absolute positioning problems? BTW: view source is also quite broken on this page. Please transfer to rickg when done for that.
Assignee: troy → rickg
Component: Style System → Parser
There are at least two parser problems that are causing things to get screwed up. The first problem is that DIV tags aren't nesting like they should. Here's a small subset of the HTML that demonstrates that problem. The second problem can be seen if you remove the FORM tag enclosing the SELECT element. That causes all of the various OPTION text to just become text inside of the second DIV Here's the HTML: <html> <head><title>HotBot</title> <style> #text2 {font-family:Verdana,Arial,Geneva; font-size:12px} #searchFont {font-family:courier,new-courier,monospace; font-size:11px} .searchContent{ top:59px; left:0; width:202px; position:absolute; border-color:#FF0000 ; border-right-style:solid; border-right-width:3px; background-color:#99FF33; layer-background-color:#99FF33; z-index:10; padding-left:6px; float:left; } .redBox1{ font-size:14px; color:#FFFFFF; border:1px none red; layer-background-color:#FF0000; background-color:#FF0000; width:175px; padding-bottom:2px; } </style> </head> <body text=#000000 link=#3333CC vlink=#660099 bgcolor=#FFFFFF> <div CLASS="searchContent" ID="searchFont"> <div ID="text2"> <form> <SELECT name="SM" > <OPTION value="MC" selected>all the words <OPTION value="SC">any of the words <OPTION value="phrase">exact phrase <OPTION value="title">the page title <OPTION value="name">the person <OPTION value="url">links to this URL <OPTION value="B">Boolean phrase </SELECT> </form> </div> </div> </body></html>
Status: NEW → ASSIGNED
This is completed fixed yet, but it's much better.
Assignee: rickg → peterl
Status: ASSIGNED → NEW
Peter, I think this is yours now. CSS appears to be coming through. Also, can you comment on what you saw with view-source? It looks pretty good to me.
Component: Parser → Style System
Peter, I think this is yours now. CSS appears to be coming through. Also, can you comment on what you saw with view-source? It looks pretty good to me.
Assignee: peterl → troy
Still some positioning problems. I think the style data is OK. Still problems with view source. I sent seperate data to rickg on that.
Assignee: troy → rickg
Component: Style System → Parser
There's still at least one problem with the parser. And that INPUT element in the upper-left corner looks like a parser problem as well... Case #1: <html> <head><style> .redBox2{ color:#FFFFFF; border: 1px none red; layer-background-color:#FF0000; background-color:#FF0000; padding:2px; padding-right:3px; width:10px; } </style></head> <body> <div CLASS="redBox2" ID="text2"> <input type="SUBMIT" name="submit" value="SEARCH" ID="searchFont"> </div> </body> </html>
Status: NEW → ASSIGNED
Fixed by improvements to new DTD system; also fixed one in comment parsing.
petersen, could you please Verify this today? Thanks!
Status: RESOLVED → VERIFIED
This problem is fixed in the Thurs Jan 21 build.
Inserting Milestone info.
You need to log in before you can comment on or make changes to this bug.