[PATCH] khtml_part.cpp: missing end tag in messagebox

[Melchior FRANZ]

Hi,

I just came across an error message with missing </qt> tag. Or should
the preceding <qt> be deleted instead? Unfortunately, this doesn't seem
to be documented, at least not in [qk]messagebox.{cpp,h}. And there are
no reliable reference files. Some people use surrounding <qt> tags for
messages with HTML code, others only use <p>..</p> (for example
kdelibs/kdeui/tests/kmessageboxtest.cpp) and the rest happily uses HTML
in messages without any surrounding tag. What's the right thing to do?

m.



Index: khtml_part.cpp
===================================================================
RCS file: /home/kde/kdelibs/khtml/khtml_part.cpp,v
retrieving revision 1.800
diff -u -p -r1.800 khtml_part.cpp
--- khtml_part.cpp	1 Feb 2003 15:43:29 -0000	1.800
+++ khtml_part.cpp	1 Feb 2003 18:25:56 -0000
@@ -4977,7 +4977,7 @@ bool KHTMLPart::checkLinkSecurity(const 
     else
     {
 	    KMessageBox::error( 0,
-				i18n( "<qt>This untrusted page contains a link<BR><B>%1</B><BR>to your local file system.").arg(linkURL.url()),
+				i18n( "<qt>This untrusted page contains a link<BR><B>%1</B><BR>to your local file system.</qt>").arg(linkURL.url()),
 				i18n( "Security Alert" ));
     }