Closed Bug 1933 Opened 26 years ago Closed

table displays incorrectly - <table> closed prematurely

Categories

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

x86
Windows 95
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: wtpooh, Assigned: rickg)

References

()

Details

When viewing http://summerbridge.stanford.edu with 12-15-98 build from Mozilla.org, the tables are messed up. It should be a simple 2x2 table, but a <hr> in the lower right cell moves itself and everything below it in that cell outside of the table: here's what it should look like: ----------------------- aa aaaa aaa|bb bbbb bb| ----------------------- ccc ccc cc | dd ddd | ccc cc ccc | <hr> | | dd ddd d | | <hr> | | dd dd ddd| ----------------------- here's what it does look like: ----------------------- aa aaaa aaa|bb bbbb bb| ----------------------- ccc ccc cc | dd ddd | ccc cc ccc | <hr> | | dd ddd d | ----------------------- <hr> dd dd ddd
Assignee: buster → rickg
Severity: minor → normal
Component: HTMLTables → Parser
Summary: table displays incorrectly - <hr> seems to create a new row → table displays incorrectly - <table> closed prematurely
reassigned to rickg, component->parser, severity->normal, summary changed to reflect the problem more precisely. In case the URL source changes, I have the copied the original source document below. =================== test case ===================== <HTML> <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN"> <!--This file created 1:04 AM 11/19/98 by Claris Home Page version 3.0--> <HEAD> <TITLE>Summerbridge National</TITLE> <META NAME=GENERATOR CONTENT="Claris Home Page 3.0"> <X-CLARIS-WINDOW TOP=27 BOTTOM=432 LEFT=17 RIGHT=547> <X-CLARIS-TAGVIEW MODE=minimal> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"><!--This file created 11:19 PM 9/10/98 by Claris Home Page version 3.0--> </HEAD> <BODY BGCOLOR="#FFFFFF"> <P><TABLE BORDER=0 BGCOLOR="#FFFFFF" CELLSPACING=5 CELLPADDING=5> <TR> <TD> <P><A HREF="http://summerbridge.stanford.edu:1997"><IMG SRC="Images/apply.gif" WIDTH=165 HEIGHT=136 BORDER=0 ALIGN=middle></A></P> </TD> <TD ALIGN=center> <H1><FONT SIZE="+3" COLOR="#FF0000">Summerbridge National</FONT></H1> <P ALIGN=right><FONT COLOR="#FF0000"><EM>Students Teaching Students</EM></FONT></P> <P><FONT SIZE="-1" COLOR="#FF0000"><EM>361 Oak Street, San Francisco, CA 94102</EM></FONT><FONT COLOR="#FF0000"><EM> </EM></FONT></P> <P><FONT SIZE="-1" COLOR="#FF0000"><EM>PHONE: (415) 865-2976 FAX: (415) 865-2979 EMAIL: </EM></FONT><A HREF="mailto:sbnation@aol.com"><FONT SIZE="-1" COLOR="#FF0000"><EM>sbnation@aol.com</EM></FONT></A><FONT SIZE="-2" COLOR="#FF0000"><EM> </EM></FONT></P> <ADDRESS><FONT SIZE="-2" COLOR="#FF0000"><EM><IMG SRC="Images/AmeriCorpsLogo.jpg" WIDTH=90 HEIGHT=90 ALIGN=right hspace=0></EM></FONT></ADDRESS> <P>&nbsp;</P> <ADDRESS><FONT SIZE="-2" COLOR="#FF0000"><EM>An AmeriCorps Program</EM></FONT></ADDRESS> <P></P> </TD> </TR> <TR> <TD VALIGN=top> <P><A HREF="http://summerbridge.stanford.edu/program.html"><IMG SRC="Images/program.gif" ALT="The SB Program" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/national.html"><IMG SRC="Images/national.gif" ALT="About National" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/students.html"><IMG SRC="Images/studentsNav.gif" ALT="The Students" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/teachers.html"><IMG SRC="Images/teachersNav.gif" ALT="The Teachers" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/tasb.htm"><IMG SRC="Images/tasbbtn.jpg" WIDTH=145 HEIGHT=41 X-CLARIS-USEIMAGEWIDTH X-CLARIS-USEIMAGEHEIGHT BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/americorps.html"><IMG SRC="Images/americorps.gif" ALT="SB Americorps" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/friends.html"><IMG SRC="Images/friends.gif" ALT="Friends of SBN" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="https://www2.hostdepot.com/SSL/givedirect.org/givefrm.asp?Action=LI&CID=65 7"><IMG SRC="Images/give.GIF" WIDTH=145 HEIGHT=41 X-CLARIS-USEIMAGEWIDTH X-CLARIS-USEIMAGEHEIGHT BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/Inside/index.html"><IMG SRC="Images/inside_sm.gif" ALT="Inside SB" WIDTH=145 HEIGHT=73 BORDER=0 ALIGN=bottom></A></P> </TD> <TD VALIGN=top> <H2><CENTER><FONT COLOR="#FF0000">We're back!</FONT></CENTER></H2> <P>Yes, this site was down from about 11/1 to 11/9 due to construction at Stanford. In a few weeks we will be moving to <A HREF="http://www.summerbridge.org">www.summerbridge.org</A> , so get ready for the big (and permanent :-) ) shift!</P> <P>&nbsp;</P> <P> <HR color="#FFFF00"> <FONT SIZE="-1" COLOR="#000000"><STRONG>Welcome to Summerbridge National's web page. This site is designed to share with you information on Summerbridge National and the amazing students and teachers that make up the program, as well as to provide you with answers to some common questions. Summerbridge National is a national non-profit organization, dedicated to promoting all students as learners, teachers and leaders.</STRONG></FONT><STRONG> </STRONG></P> <P> <HR color="#FFFF00"> </P> <UL type=square> <LI><A HREF="PROGRAM.HTML"><FONT SIZE="+1">The Summerbridge Program</FONT></A> <P type=square>"Ownership of this program belongs to all the students who have worked and/or studied at Summerbridge over the years. Each and every student has left an idea behind, an event to continue, a plan to modify, a success to celebrate, a risk worth taking, a criticism well earned. Since a workshop connotes a work in progress, Summerbridge can never be a finished product." -<EM>Lois Loofbourrow, Founder, Summerbridge National</EM></P></LI> <LI><A HREF="national.html"><FONT SIZE="+1">Summerbridge National: Who We Are </FONT></A> <P type=square>"Getting younger students into learning and older students into teaching." -<EM> Newsweek</EM></P></LI> <LI><A HREF="students.html"><FONT SIZE="+1">Celebrating Our Students</FONT></A><FONT SIZE="+1"> </FONT> <P type=square>"All of the students at Summerbridge have this special chance to learn so many things." <EM>-Summerbridge Student</EM></P></LI> <LI><A HREF="teachers.html"><FONT SIZE="+1">Celebrating Our Teachers</FONT></A> <P type=square>"This program gave me so much. It asked me to give and I did because the rewards were so huge. I've never felt so tied into people as I did to my students. For six weeks, I watched them struggle and grow and stretch to understand the material--and I applauded wildly when they did reach and surpass that barrier of understanding." -<EM>Summerbridge Portland Teacher</EM></P></LI> <LI><A HREF="americor.html"><FONT SIZE="+1">Summerbridge AmeriCorps</FONT></A> <P type=square>"So many communities need help today. And so many young people want to contribute... AmeriCorps is an opportunity to bring these two great needs together." -<EM>Janet Reno, United States Attorney General</EM></P></LI> <LI><A HREF="friends.html"><FONT SIZE="+1">Friends of Summerbridge </FONT></A> <P type=square>"I believe that in my long years as a teacher, an educator and a superintendant that I have not seen a better program for many of our minority under-represented youth." -<EM>Ramon Cortines, Former United States Undersecretary of Education</EM></P></LI> <LI><A HREF="Inside/"><FONT SIZE="+1">Inside Summerbridge</FONT></A> <P type=square>Please visit our student lounge and take a look at some of our students' work.</P></LI> </UL> </TD> </TR> <TR> <TD> <P>&nbsp;</P> </TD> <TD> <CENTER><TABLE BORDER=3 CELLSPACING=5 CELLPADDING=5> <TR> <TD ALIGN=center> <P><A HREF="mailto:sbnation@aol.com"><STRONG>Contacting SBN</STRONG></A></P> </TD> <TD ALIGN=center> <P><A HREF="shortcut.html"><STRONG>Shortcuts</STRONG></A></P> </TD> <TD ALIGN=center> <P><STRONG>Home Page</STRONG></P> </TD> </TR> </TABLE> </CENTER> </TD> </TR> <TR> <TD> <P>&nbsp;</P> </TD> <TD> <P ALIGN=right><FONT SIZE="-1">This page was last updated: November 19, 1998</FONT></P> </TD> </TR> </TABLE> </P> </BODY> </HTML>
reassigned to rickg, component->parser, severity->normal, summary changed to reflect the problem more precisely. In case the URL source changes, I have the copied the original source document below. =================== test case ===================== <HTML> <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN"> <!--This file created 1:04 AM 11/19/98 by Claris Home Page version 3.0--> <HEAD> <TITLE>Summerbridge National</TITLE> <META NAME=GENERATOR CONTENT="Claris Home Page 3.0"> <X-CLARIS-WINDOW TOP=27 BOTTOM=432 LEFT=17 RIGHT=547> <X-CLARIS-TAGVIEW MODE=minimal> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"><!--This file created 11:19 PM 9/10/98 by Claris Home Page version 3.0--> </HEAD> <BODY BGCOLOR="#FFFFFF"> <P><TABLE BORDER=0 BGCOLOR="#FFFFFF" CELLSPACING=5 CELLPADDING=5> <TR> <TD> <P><A HREF="http://summerbridge.stanford.edu:1997"><IMG SRC="Images/apply.gif" WIDTH=165 HEIGHT=136 BORDER=0 ALIGN=middle></A></P> </TD> <TD ALIGN=center> <H1><FONT SIZE="+3" COLOR="#FF0000">Summerbridge National</FONT></H1> <P ALIGN=right><FONT COLOR="#FF0000"><EM>Students Teaching Students</EM></FONT></P> <P><FONT SIZE="-1" COLOR="#FF0000"><EM>361 Oak Street, San Francisco, CA 94102</EM></FONT><FONT COLOR="#FF0000"><EM> </EM></FONT></P> <P><FONT SIZE="-1" COLOR="#FF0000"><EM>PHONE: (415) 865-2976 FAX: (415) 865-2979 EMAIL: </EM></FONT><A HREF="mailto:sbnation@aol.com"><FONT SIZE="-1" COLOR="#FF0000"><EM>sbnation@aol.com</EM></FONT></A><FONT SIZE="-2" COLOR="#FF0000"><EM> </EM></FONT></P> <ADDRESS><FONT SIZE="-2" COLOR="#FF0000"><EM><IMG SRC="Images/AmeriCorpsLogo.jpg" WIDTH=90 HEIGHT=90 ALIGN=right hspace=0></EM></FONT></ADDRESS> <P>&nbsp;</P> <ADDRESS><FONT SIZE="-2" COLOR="#FF0000"><EM>An AmeriCorps Program</EM></FONT></ADDRESS> <P></P> </TD> </TR> <TR> <TD VALIGN=top> <P><A HREF="http://summerbridge.stanford.edu/program.html"><IMG SRC="Images/program.gif" ALT="The SB Program" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/national.html"><IMG SRC="Images/national.gif" ALT="About National" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/students.html"><IMG SRC="Images/studentsNav.gif" ALT="The Students" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/teachers.html"><IMG SRC="Images/teachersNav.gif" ALT="The Teachers" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/tasb.htm"><IMG SRC="Images/tasbbtn.jpg" WIDTH=145 HEIGHT=41 X-CLARIS-USEIMAGEWIDTH X-CLARIS-USEIMAGEHEIGHT BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/americorps.html"><IMG SRC="Images/americorps.gif" ALT="SB Americorps" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/friends.html"><IMG SRC="Images/friends.gif" ALT="Friends of SBN" WIDTH=145 HEIGHT=41 BORDER=0 ALIGN=bottom></A></P> <P><A HREF="https://www2.hostdepot.com/SSL/givedirect.org/givefrm.asp?Action=LI&CID=65 7"><IMG SRC="Images/give.GIF" WIDTH=145 HEIGHT=41 X-CLARIS-USEIMAGEWIDTH X-CLARIS-USEIMAGEHEIGHT BORDER=0 ALIGN=bottom></A></P> <P><A HREF="http://summerbridge.stanford.edu/Inside/index.html"><IMG SRC="Images/inside_sm.gif" ALT="Inside SB" WIDTH=145 HEIGHT=73 BORDER=0 ALIGN=bottom></A></P> </TD> <TD VALIGN=top> <H2><CENTER><FONT COLOR="#FF0000">We're back!</FONT></CENTER></H2> <P>Yes, this site was down from about 11/1 to 11/9 due to construction at Stanford. In a few weeks we will be moving to <A HREF="http://www.summerbridge.org">www.summerbridge.org</A> , so get ready for the big (and permanent :-) ) shift!</P> <P>&nbsp;</P> <P> <HR color="#FFFF00"> <FONT SIZE="-1" COLOR="#000000"><STRONG>Welcome to Summerbridge National's web page. This site is designed to share with you information on Summerbridge National and the amazing students and teachers that make up the program, as well as to provide you with answers to some common questions. Summerbridge National is a national non-profit organization, dedicated to promoting all students as learners, teachers and leaders.</STRONG></FONT><STRONG> </STRONG></P> <P> <HR color="#FFFF00"> </P> <UL type=square> <LI><A HREF="PROGRAM.HTML"><FONT SIZE="+1">The Summerbridge Program</FONT></A> <P type=square>"Ownership of this program belongs to all the students who have worked and/or studied at Summerbridge over the years. Each and every student has left an idea behind, an event to continue, a plan to modify, a success to celebrate, a risk worth taking, a criticism well earned. Since a workshop connotes a work in progress, Summerbridge can never be a finished product." -<EM>Lois Loofbourrow, Founder, Summerbridge National</EM></P></LI> <LI><A HREF="national.html"><FONT SIZE="+1">Summerbridge National: Who We Are </FONT></A> <P type=square>"Getting younger students into learning and older students into teaching." -<EM> Newsweek</EM></P></LI> <LI><A HREF="students.html"><FONT SIZE="+1">Celebrating Our Students</FONT></A><FONT SIZE="+1"> </FONT> <P type=square>"All of the students at Summerbridge have this special chance to learn so many things." <EM>-Summerbridge Student</EM></P></LI> <LI><A HREF="teachers.html"><FONT SIZE="+1">Celebrating Our Teachers</FONT></A> <P type=square>"This program gave me so much. It asked me to give and I did because the rewards were so huge. I've never felt so tied into people as I did to my students. For six weeks, I watched them struggle and grow and stretch to understand the material--and I applauded wildly when they did reach and surpass that barrier of understanding." -<EM>Summerbridge Portland Teacher</EM></P></LI> <LI><A HREF="americor.html"><FONT SIZE="+1">Summerbridge AmeriCorps</FONT></A> <P type=square>"So many communities need help today. And so many young people want to contribute... AmeriCorps is an opportunity to bring these two great needs together." -<EM>Janet Reno, United States Attorney General</EM></P></LI> <LI><A HREF="friends.html"><FONT SIZE="+1">Friends of Summerbridge </FONT></A> <P type=square>"I believe that in my long years as a teacher, an educator and a superintendant that I have not seen a better program for many of our minority under-represented youth." -<EM>Ramon Cortines, Former United States Undersecretary of Education</EM></P></LI> <LI><A HREF="Inside/"><FONT SIZE="+1">Inside Summerbridge</FONT></A> <P type=square>Please visit our student lounge and take a look at some of our students' work.</P></LI> </UL> </TD> </TR> <TR> <TD> <P>&nbsp;</P> </TD> <TD> <CENTER><TABLE BORDER=3 CELLSPACING=5 CELLPADDING=5> <TR> <TD ALIGN=center> <P><A HREF="mailto:sbnation@aol.com"><STRONG>Contacting SBN</STRONG></A></P> </TD> <TD ALIGN=center> <P><A HREF="shortcut.html"><STRONG>Shortcuts</STRONG></A></P> </TD> <TD ALIGN=center> <P><STRONG>Home Page</STRONG></P> </TD> </TR> </TABLE> </CENTER> </TD> </TR> <TR> <TD> <P>&nbsp;</P> </TD> <TD> <P ALIGN=right><FONT SIZE="-1">This page was last updated: November 19, 1998</FONT></P> </TD> </TR> </TABLE> </P> </BODY> </HTML>
Here's a simple test case that demonstrates the same behaviour (derived from the original page). Note that <P> is wrapped around <TABLE>; later on, <HR> "orphans" a paragraph close </P>, and this then causes the table bustage. (Is this a clear explanation?). Removing these 'orpaned' /P's cleans from the original page clears up the problem (there two of them about line 86 near 'learners, teachers, and leaders'). <HTML><HEAD></HEAD><BODY> <P><TABLE BORDER=1 CELLSPACING=5 CELLPADDING=5> <TR><TD>foo</TD><TD>foo</TD></TR> <TR> <TD>foo</TD> <TD> <!-- the HR (below) closes the P; then the /P closes the P that encloses the entire table; --> <P>something <HR> </P> whatever </TD> </TR> </TABLE> </P> </BODY></HTML>
All fixed with last (big) update to parsing engine.
QA Contact: 4120
URL updated to reflect moved page. From .edu to .org. Also, bug regressed and fix Verified on 99081 win95 build.
You need to log in before you can comment on or make changes to this bug.