[freeside-commits] freeside/httemplate/elements popup_link_onclick.html, 1.1.2.2, 1.1.2.3

[Ivan,,,]

Update of /home/cvs/cvsroot/freeside/httemplate/elements
In directory wavetail.420.am:/tmp/cvs-serv16539/elements

Modified Files:
      Tag: FREESIDE_1_7_BRANCH
	popup_link_onclick.html 
Log Message:
clean up overlib usage for customer notes

Index: popup_link_onclick.html
===================================================================
RCS file: /home/cvs/cvsroot/freeside/httemplate/elements/popup_link_onclick.html,v
retrieving revision 1.1.2.2
retrieving revision 1.1.2.3
diff -u -d -r1.1.2.2 -r1.1.2.3
--- popup_link_onclick.html	31 Mar 2008 21:11:03 -0000	1.1.2.2
+++ popup_link_onclick.html	31 Mar 2008 23:08:01 -0000	1.1.2.3
@@ -25,7 +25,7 @@
 % }
 <%init>
 
-my($action, $actionlabel) = ( '', '' );
+my( $action, $actionlabel, $frame ) = ( '', '', '' );
 my( $width, $height ) = ( 540, 336 );
 my $closetext = 'Close';
 my $color = '#333399';
@@ -45,11 +45,18 @@
 $height      = $params->{'height'}      if exists $params->{'height'};
 $color       = $params->{'color'}       if exists $params->{'color'};
 $closetext   = $params->{'closetext'}   if exists $params->{'closetext'};
+$frame       = $params->{'frame'}       if exists $params->{'frame'};
 
 #stupid safari is caching the "location" of popup iframs, and submitting them
 #instead of displaying them.  this should prevent that.
 my $popup_name = 'popup-'.time. "-$$-". rand() * 2**32;
 
-my $onclick = "overlib( OLiframeContent('$action', $width, $height, '$popup_name' ), CAPTION, '$actionlabel', STICKY, AUTOSTATUSCAP, MIDX, 0, MIDY, 0, DRAGGABLE, CLOSECLICK, BGCOLOR, '$color', CGCOLOR, '$color', CLOSETEXT, '$closetext' ); return false;";
+my $onclick =
+  "overlib( OLiframeContent('$action', $width, $height, '$popup_name' ), ".
+    "CAPTION, '$actionlabel', STICKY, AUTOSTATUSCAP, MIDX, 0, MIDY, 0, ".
+    "DRAGGABLE, CLOSECLICK, ".
+    "BGCOLOR, '$color', CGCOLOR, '$color', CLOSETEXT, '$closetext'".
+    ( $frame ? ", FRAME, $frame" : '' ).
+  "); return false;";
 
 </%init>