Re: [evolution-patches] [gtkhtml] pre tags around table fix



On Thu, 2004-08-26 at 19:02 -0500, Larry Ewing wrote:
ok after looking at it a bit more and ignoring the monospace issue, this
patch should be replaced by the one I've attached, which does
essentially the same thing but does it when a cell starts not when the
table starts because content between the <table> and <td> tags should
still be pre.  The table parser does special things already to make make
content like that work properly and there is no need to change that.

On the other hand the patch as sent is ok to go into this point release
it won't break valid html and we can just apply this one after.
OK, I committed the original patch. I am not sure if we want to follow mozilla there. Even your patch renders differently from mozilla. Mozilla indeed keeps line breaks in text but eats line breaks around tags as in non pre mode. I looked into specification and didn't find anything about this. I am attaching my test case.

R.

<PRE>
<TABLE CELLSPACING="0" CELLPADDING="0" WIDTH="100%" border=1>
text between TABLE and TR/TD
new line
new line
<TR>
<TD>
-- <BR>
<TABLE CELLSPACING="6" border=1>
<TR>
<TD>
<IMG SRC="cid:1092190387.4553.28.camel@lostzed.mmc.com.au" WIDTH="48"
HEIGHT="48" ALIGN="top" ALT="" BORDER="0">
</TD>
<TD>
<B>Michael Zucchi</B> &lt;<A
HREF="mailto:notzed ximian com">notzed ximian com</A>&gt;<BR>
<I>&quot;born to die, live to work, it's all downhill from


here&quot;</I><BR>
<TT>Novell's <A
HREF="http://codeblogs.ximian.com/blogs/evolution/";>Evolution</A> and <A
HREF="http://www.gnu.org/philosophy/free-sw.html";>Free Software</A>
Developer</TT>
</TD>
</TR>
</TABLE>
</TD>
</TR>
</TABLE>
</PRE>
post pre


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]