Revert "Das (linksseitige) HTML-Menü komaptibel zum Textbrowser Links gemacht."
authorSven Schöling <s.schoeling@linet-services.de>
Mon, 27 Sep 2010 09:08:30 +0000 (11:08 +0200)
committerSven Schöling <s.schoeling@linet-services.de>
Mon, 27 Sep 2010 09:08:30 +0000 (11:08 +0200)
This reverts commit d2d63d80de28c7356e81eb1143d729e4415cd1b4.

Das klappt so nicht. kopf.pl kann durchaus auch alleine aufgerufen werden ohne menu einzubinden, und selbst wenn darf framesize nicht einfach als globale Variable rumlungern. Siehe Dokumentation doc/INSTALL.fcgi (Entwicklungsaspekte) und Doku in SL/Dispatcher.pm Zeile 71.

bin/mozilla/kopf.pl
bin/mozilla/menu.pl

index ad0b817..ac3935d 100644 (file)
@@ -111,24 +111,19 @@ window.onload=clockon
 my $framesize = _calc_framesize(); # framesize calculation as in menu.pl is only a quick and dirty hack here
                                    # it would be better to use a global or config variable
 
-# is there any better solution then the JS-Link for Back? Possibly with the callback variable?
-
 print qq|
 <body bgcolor="#ffffff" text="#ffffff" link="#ffffff" vlink="#ffffff" alink="#ffffff" topmargin="0" leftmargin="0"  marginwidth="0" marginheight="0" style="background-image: url('image/fade.png'); background-repeat:repeat-x;">
 <script language='JavaScript' src='js/switchmenuframe.js'></script>
 <table border="0" width="100%" background="image/bg_titel.gif" cellpadding="0" cellspacing="0">
-  <tr>|;
-   if ($framesize ne 240) {    # only show the the links in case of "links" in $ENV{HTTP_USER_AGENT} (see sub _calc_framesize)
-      print qq|
-      <td  style="color:white; font-family:verdana,arial,sans-serif; font-size: 12px;" nowrap>
+  <tr>
+    <td  style="color:white; font-family:verdana,arial,sans-serif; font-size: 12px;" nowrap>
       [<a href="JavaScript:Switch_Menu(|.$framesize.qq|);" title="| . $locale->text('Switch Menu on / off') . qq|">| . $locale->text('Menu') . qq|</a>]
       &nbsp;[<a HREF="login.pl" target="_blank" title="| . $locale->text('Open a further Lx-Office Window or Tab') . qq|">| . $locale->text('New Win/Tab') . qq|</a>]
       &nbsp;[<a href="JavaScript:top.main_window.print();" title="| . $locale->text('Hardcopy') . qq|">| . $locale->text('Print') . qq|</a>]
       &nbsp;[<a href="Javascript:top.main_window.history.back();" title="| . $locale->text('Go one step back') . qq|">| . $locale->text('Back') . qq|</a>]
       &nbsp;[<a href="Javascript:top.main_window.history.forward();" title="| . $locale->text('Go one step forward') . qq|">| . $locale->text('Fwd') . qq|</a>]
-      </td>|;
-   }
-   print qq|
+      <!-- is there any better solution for Back? Possibly with the callback variable? -->
+    </td>
     <td align="right" style="vertical-align:middle; color:white; font-family:verdana,arial,sans-serif; font-size: 12px;" nowrap>|
   . $login . $datum . qq| <script>writeclock()</script>&nbsp;
     </td>
index f6305df..115880d 100644 (file)
@@ -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 one click 
 #               JS switchable HTML-menu - Sven Donath <lxo@dexo.de>
 #######################################################################
 
@@ -45,7 +45,6 @@ use URI;
 
 my $menufile = "menu.ini";
 my $mainlevel;
-our $framesize;
 
 # end of main
 
@@ -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(); # how to get it into kopf.pl or vice versa?
 
   $mainlevel = $form->{level};
   $mainlevel =~ s/\Q$mainlevel\E--//g;
@@ -98,7 +97,7 @@ sub acc_menu {
 
 |;
   print qq|<div align="left">\n<table width="|
-    . ($framesize-2)
+    . $framesize
     . qq|" border="0">\n|;
 
   &section_menu($menu);
@@ -200,20 +199,18 @@ sub section_menu {
           if ($zeige) {
             if (scalar @chunks <= 1) {
               print
-                qq|<tr><td class="hover" height="16" >$spacer|
+                qq|<tr><td class="hover" height="16" >$spacer| 
                 . $menu->menuitem(\%myconfig, \%$form, $item, $level) ;
-
-            if (-f "image/icons/16x16/$label_icon" && ($framesize ne 240))
-             { print
-                qq|<img src="image/icons/16x16/$label_icon" border="0" style="vertical-align:text-top" title="|
-                . $label
-                . qq|">&nbsp;&nbsp;| }
+              
+            if (-f "image/icons/16x16/$label_icon")
+             { 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) {
-                    print qq|<img src="image/unterpunkt.png" border="0" style="vertical-align:text-top">|;
-                   }
+               print qq|<img src="image/unterpunkt.png" border="0" style="vertical-align:text-top">|;   
                 }
-
+                
                print
                  qq|$label</a></td></tr>\n|;
             } else {
@@ -234,12 +231,7 @@ sub section_menu {
       } else {
         my $ml_ = $form->escape($ml);
         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) {
-                  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|;
+          qq|<tr><td class="bg" height="24" align="left" valign="middle"><a href="menu.pl?action=acc_menu&level=$ml_" class="nohover" title="$label"><img src="image/icons/24x24/$item.png" border="0" style="vertical-align:middle" title="$label">&nbsp;$label</a>&nbsp;&nbsp;&nbsp;&nbsp;</td></tr>\n|;
         &section_menu($menu, $item);
 
         print qq|\n|;