[imp] problem in message template ?

Rémi Cohen-Scali Remi@Cohen-Scali.com
Tue, 18 Dec 2001 23:06:02 +0100


Jon Parise wrote:

>On Tue, Dec 18, 2001 at 09:46:49PM +0000, Michael Cochrane wrote:
>
>>On the template thing, i spotted one error last night. I thinks
>>it's in the mailbox common header. the search <link> tag is not
>>closed with /> as it should be with xhtml. All other <link>
>>tages are correct :-)
>>
> 
>I'm glad you spotted that.  Fixed in cvs.
>
BTW about header, the html tag ('<html>') could contains the language 
spec as in <html lang="fr-FR">.

diff -urNwbB imp/templates/common-header.inc imp-new/templates/common-header.inc
--- imp/templates/common-header.inc    Tue Dec 18 13:34:48 2001
+++ imp-new/templates/common-header.inc    Fri Dec 14 02:51:05 2001
@@ -10,7 +10,13 @@
 <!-- Horde Project: http://horde.org/ | IMP: http://horde.org/imp/ -->
 <!--    GNU Public License: http://www.fsf.org/copyleft/gpl.html   -->
 
-<html>
+<?php
+if (isset($language) && !empty($language)) {
+    echo "<html lang=\"".preg_replace("/_/", "-", $language)."\">\n";
+} else {
+    echo "<html>\n";
+}
+?>
 <head>
 <?php

 
what do you think ?

-- 
                                                          -o)
                   Remi Cohen-Scali                       /\\
<Remi@Cohen-Scali.com>          <rcoscali@rcsnet.net>    _\_v
                                                         ----