Merge remote branch 'refs/remotes/origin/master'

[kivitendo-erp.git] / bin / mozilla / menu.pl
diff --git a/bin/mozilla/menu.pl b/bin/mozilla/menu.pl

index f6305df..ae2d6b7 100644 (file)
--- a/bin/mozilla/menu.pl
+++ b/bin/mozilla/menu.pl
@@ -33,7 +33,7 @@
  # CHANGE LOG:
  #   DS. 2002-03-25  Created
  #  2004-12-14 - New Optik - Marco Welter <mawe@linux-studio.de>
-#  2010-08-19 - Icons for sub entries and one click
+#  2010-08-19 - Icons for sub entries and single click behavior, unlike XUL-Menu
  #               JS switchable HTML-menu - Sven Donath <lxo@dexo.de>
  #######################################################################
  
@@ -45,19 +45,18 @@ use URI;
  
  my $menufile = "menu.ini";
  my $mainlevel;
-our $framesize;
  
  # end of main
  
  sub display {
    $main::lxdebug->enter_sub();
  
-  my $form     = $main::form;
+  my $form      = $main::form;
  
-  my $callback   = $form->unescape($form->{callback});
-  $callback      = URI->new($callback)->rel($callback) if $callback;
-  $callback      = "login.pl?action=company_logo"      if $callback =~ /^(\.\/)?$/;
-  my $framesize  = _calc_framesize();
+  my $callback  = $form->unescape($form->{callback});
+  $callback     = URI->new($callback)->rel($callback) if $callback;
+  $callback     = "login.pl?action=company_logo"      if $callback =~ /^(\.\/)?$/;
+  my $framesize = _calc_framesize();
  
    $form->header;
  
@@ -83,7 +82,7 @@ sub acc_menu {
  
    my $form      = $main::form;
    my $locale    = $main::locale;
-     $framesize = _calc_framesize(); # how to get it into kopf.pl or vice versa?
+  my $framesize = _calc_framesize();
  
    $mainlevel = $form->{level};
    $mainlevel =~ s/\Q$mainlevel\E--//g;
@@ -95,8 +94,8 @@ sub acc_menu {
  
    print qq|
  <body class="menu">
-
-|;
+ 
+|; 
    print qq|<div align="left">\n<table width="|
      . ($framesize-2)
      . qq|" border="0">\n|;
@@ -116,9 +115,10 @@ sub section_menu {
    $main::lxdebug->enter_sub();
    my ($menu, $level) = @_;
  
-  my $form     = $main::form;
-  my %myconfig = %main::myconfig;
-  my $locale   = $main::locale;
+  my $form      = $main::form;
+  my %myconfig  = %main::myconfig;
+  my $locale    = $main::locale;
+  my $is_not_links_browser = $ENV{HTTP_USER_AGENT} =~ /links/i ? 0 : 1;
  
    my $zeige;
  
@@ -203,13 +203,13 @@ sub section_menu {
                  qq|<tr><td class="hover" height="16" >$spacer|
                  . $menu->menuitem(\%myconfig, \%$form, $item, $level) ;
  
-            if (-f "image/icons/16x16/$label_icon" && ($framesize ne 240))
+            if (-f "image/icons/16x16/$label_icon" && ($is_not_links_browser))
               { print
                  qq|<img src="image/icons/16x16/$label_icon" border="0" style="vertical-align:text-top" title="|
                  . $label
                  . qq|">&nbsp;&nbsp;| }
              else {
-                   if ($framesize ne 240) {
+                   if ($is_not_links_browser) {
                      print qq|<img src="image/unterpunkt.png" border="0" style="vertical-align:text-top">|;
                     }
                  }
@@ -236,7 +236,7 @@ sub section_menu {
          print
            qq|<tr><td class="bg" height="24" align="left" valign="middle">
            <a href="menu.pl?action=acc_menu&level=$ml_" class="nohover" title="$label">|;
-              if ($framesize ne 240) {
+              if ($is_not_links_browser) {
                    print qq|<img src="image/icons/24x24/$item.png" border="0" style="vertical-align:middle" title="$label">|;
                }
            print qq|&nbsp;$label</a>&nbsp;&nbsp;&nbsp;&nbsp;</td></tr>\n|;
@@ -256,7 +256,7 @@ sub _calc_framesize {
  
    return  $is_mobile_browser && $is_mobile_style ?  130
          : $is_lynx_browser                       ?  240
-        :                                           180;
+        :                                           200;
  }
  
  1;