Some browsers may complain when fomr not present. Corrected.
[squirrelmail.git] / functions / page_header.php
index 9e95b3b364f6ac63f1ffff24e3f02e257a9e36df..0e204eb709c104a66981fc2678301e411ad8e839 100644 (file)
@@ -7,39 +7,19 @@
     **  $Id$
     **/
 
-   session_start();
+   if (defined('page_header_php'))
+       return;
+   define('page_header_php', true);
 
-   $page_header_php = true;
-
-   if (!isset($prefs_php))
-      include ("../functions/prefs.php");
-   if (!isset($i18n_php))
-      include ("../functions/i18n.php");
-   if (!isset($plugin_php))
-      include ("../functions/plugin.php");
-
-   // Check to see if gettext is installed
-   $headers_sent=set_up_language(getPref($data_dir, $username, "language"));
-
-   // This is done to ensure that the character set is correct.
-   // But first checks whether we have already sent headers
-   // with charset when we were setting up the user language.
-   // Otherwise user ends up with the default charset overriding
-   // his selected one.
-   if (!$headers_sent && $default_charset != "")
-      header ("Content-Type: text/html; charset=$default_charset");
+   // Always set up the language before calling these functions
 
    function displayHtmlHeader ($title="SquirrelMail") {
      global $theme_css;
 
-      echo '<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">';
-      echo "\n\n";
-      echo "<HTML>\n";
-      echo "<HEAD>\n";
-      if ($theme_css != "") {
-        printf ('<LINK REL="stylesheet" TYPE="text/css" HREF="%s">', 
-                $theme_css);
-        echo "\n";
+      echo '<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">' .
+           "\n\n<HTML>\n<HEAD>\n";
+      if ($theme_css != '') {
+        echo "<LINK REL=\"stylesheet\" TYPE=\"text/css\" HREF=\"$theme_css\">\n"; 
       }
       
       do_hook ("generic_header");
       echo "</HEAD>\n\n";
    }
 
-   function displayInternalLink ($path, $text, $target="") {
+   function displayInternalLink ($path, $text, $target='') {
       global $base_uri;
 
-      if ($target != "")
+      if ($target != '')
          $target = " target=\"$target\"";
       
       echo '<a href="'.$base_uri.$path.'"'.$target.'>'.$text.'</a>';
@@ -60,9 +40,7 @@
    function displayPageHeader($color, $mailbox) {
       displayHtmlHeader ();
 
-      printf('<BODY TEXT="%s" BGCOLOR="%s" LINK="%s" VLINK="%s" ALINK="%s">',
-             $color[8], $color[4], $color[7], $color[7], $color[7]);
-      echo "\n\n";
+      echo "<BODY TEXT=\"$color[8]\" BGCOLOR=\"$color[4]\" LINK=\"$color[7]\" VLINK=\"$color[7]\" ALINK=\"$color[7]\" onLoad='if ( 0 in document.forms && document.forms[0].elements[0].type == \"text\" ) { document.forms[0].elements[0].focus(); }'>\n\n";
 
       /** Here is the header and wrapping table **/
       $shortBoxName = readShortMailboxName($mailbox, ".");
@@ -78,7 +56,7 @@
       echo "</TABLE>\n\n";
       echo "<TABLE BGCOLOR=\"$color[4]\" BORDER=0 WIDTH=\"100%\" CELLSPACING=0 CELLPADDING=2>\n";
       echo "   <TR>\n";
-      echo "      <TD ALIGN=left WIDTH=\"99%\">\n";
+      echo "      <TD ALIGN=left WIDTH=\"90%\">\n";
       $urlMailbox = urlencode($mailbox);
       displayInternalLink ("src/compose.php?mailbox=$urlMailbox", _("Compose"), "right");
       echo "&nbsp;&nbsp;\n";
 
       do_hook("menuline");
 
-      echo "      </TD><TD ALIGN=right nowrap WIDTH=\"1%\">\n";
-      echo "         <A HREF=\"http://www.squirrelmail.org/\" TARGET=\"_top\">SquirrelMail</A>\n";
+      echo "      </TD><TD ALIGN=right nowrap WIDTH=\"10%\">\n";
+      echo "         <A HREF=\"http://www.squirrelmail.org/\" TARGET=\"_blank\">SquirrelMail</A>\n";
       echo "      </TD>\n";
       echo "   </TR>\n";
       echo "</TABLE>\n\n";
   }
-?>
+?>
\ No newline at end of file