[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    more HTML cleanup
From:       Daniel Naber <dnaber () mini ! gt ! owl ! de>
Date:       2000-04-24 19:13:30
[Download RAW message or body]

Hi,

here's another cleanup. With this, you can now also select the text in the 
header. Before, the HTML was like this:

<font face="something">
<div>
 bla <br>
 blah<br>
 blah
</div>
<div>
 bla <br>
 blah<br>
 blah
</div>
</font>

This was wrong, I think. Now the font tag is gone. I'm not sure if the khtml 
guys think this is a bug in theit code...

Regards
 Daniel

["html-fixes-2.diff" (text/plain)]

--- kmreaderwin.cpp_cleaned	Mon Apr 24 16:58:33 2000
+++ kmreaderwin.cpp	Mon Apr 24 20:18:28 2000
@@ -402,9 +402,6 @@
   if (mBackingPixmapOn)
     mViewer->write(" background=\"file://" + mBackingPixmapStr + "\"");
 
-  // TODO: move this to stylesheet, too:
-  mViewer->write("><font face=\"" + mBodyFont +"\">");
-
 #if defined CHARSETS
   printf("Setting viewer charset to %s\n",(const char *)mMsg->charset());
   mViewer->setCharset(mMsg->charset());
@@ -412,7 +409,7 @@
 
   parseMsg(mMsg);
 
-  mViewer->write("</font></body></html>");
+  mViewer->write("</body></html>");
   mViewer->end();
 }
 


[prev in list] [next in list] [prev in thread] [next in thread]