From b6260b4f39cf34b9e0e5b8560ef88c8881ba0766 Mon Sep 17 00:00:00 2001
From: Ivan Kohler <ivan@freeside.biz>
Date: Sun, 14 Jun 2015 13:59:11 -0700
Subject: UI spring cleaning: customer edit

---
 httemplate/edit/cust_main.cgi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'httemplate/edit/cust_main.cgi')

diff --git a/httemplate/edit/cust_main.cgi b/httemplate/edit/cust_main.cgi
index b6f6b0dea..38bae4fa8 100755
--- a/httemplate/edit/cust_main.cgi
+++ b/httemplate/edit/cust_main.cgi
@@ -40,7 +40,7 @@
 % }
 <BR>
 <TABLE> <TR>
-  <TD STYLE="width:650px">
+  <TD>
 %#; padding-right:2px; vertical-align:top">
     <FONT CLASS="fsinnerbox-title"><% mt('Billing address') |h %></FONT>
     <TABLE CLASS="fsinnerbox" WIDTH="100%">
@@ -58,7 +58,7 @@
 </TR>
 <TR><TD STYLE="height:14px"></TD></TR>
 <TR>
-  <TD STYLE="width:650px">
+  <TD>
     <FONT CLASS="fsinnerbox-title"><% mt('Service address') |h %></FONT>
     <INPUT TYPE="checkbox" 
            NAME="same"
-- 
cgit v1.2.1


From 99fa7a9f3c628a9a63983cafa131df9b331d30d6 Mon Sep 17 00:00:00 2001
From: Ivan Kohler <ivan@freeside.biz>
Date: Sun, 14 Jun 2015 14:05:53 -0700
Subject: better UI/label for flag to include service address on invoices,
 RT#29406

---
 httemplate/edit/cust_main.cgi | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

(limited to 'httemplate/edit/cust_main.cgi')

diff --git a/httemplate/edit/cust_main.cgi b/httemplate/edit/cust_main.cgi
index 38bae4fa8..069414b11 100755
--- a/httemplate/edit/cust_main.cgi
+++ b/httemplate/edit/cust_main.cgi
@@ -80,13 +80,12 @@
       &>
 % unless ($conf->exists('invoice-ship_address')) { #it's always on, so hide per-cust config
         <TR>
-          <TD>&nbsp;</TD>
-          <TD COLSPAN="7">
+          <TD COLSPAN="8">
             <% include('/elements/checkbox.html',
                  'field'      => 'invoice_ship_address',
                  'value'      => 'Y',
                  'curr_value' => $cust_main->invoice_ship_address,
-                 'postfix'    => emt('included on invoices'),
+                 'postfix'    => emt('Include service address on invoices'),
             ) %>
           </TD>
         </TR>
-- 
cgit v1.2.1


From 506de9ec7d74b35d2bed671526a4363d022026b1 Mon Sep 17 00:00:00 2001
From: Ivan Kohler <ivan@freeside.biz>
Date: Sun, 14 Jun 2015 14:25:25 -0700
Subject: re-enable calling samechanged on page load, fallout from #21327,
 #21924

---
 httemplate/edit/cust_main.cgi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'httemplate/edit/cust_main.cgi')

diff --git a/httemplate/edit/cust_main.cgi b/httemplate/edit/cust_main.cgi
index 069414b11..877d1e80e 100755
--- a/httemplate/edit/cust_main.cgi
+++ b/httemplate/edit/cust_main.cgi
@@ -108,7 +108,7 @@ function samechanged(what) {
     t1.style.visibility = 'visible'
   }
 }
-//samechanged(document.getElementById('same'));
+samechanged(document.getElementById('same'));
 </SCRIPT>
 
 <BR>
-- 
cgit v1.2.1


From 0f57e99bd0af253afa16e20f1141afee851b8d5a Mon Sep 17 00:00:00 2001
From: Ivan Kohler <ivan@freeside.biz>
Date: Sun, 14 Jun 2015 14:26:19 -0700
Subject: why wouldn't we want to hide the whole thing rather than leave a big
 blank gap?

---
 httemplate/edit/cust_main.cgi | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

(limited to 'httemplate/edit/cust_main.cgi')

diff --git a/httemplate/edit/cust_main.cgi b/httemplate/edit/cust_main.cgi
index 877d1e80e..d4129a37c 100755
--- a/httemplate/edit/cust_main.cgi
+++ b/httemplate/edit/cust_main.cgi
@@ -97,15 +97,13 @@
 
 <SCRIPT>
 function samechanged(what) {
-%# not display = 'none', because we still want it to take up space
 %#  document.getElementById('table_ship_location').style.visibility = 
-%#    what.checked ? 'hidden' : 'visible';
   var t1 = document.getElementById('table_ship_location');
   if ( what.checked ) {
-    t1.style.visibility = 'hidden';
+    t1.style.display = 'none';
   }
   else {
-    t1.style.visibility = 'visible'
+    t1.style.display = ''
   }
 }
 samechanged(document.getElementById('same'));
-- 
cgit v1.2.1


From a5ebbe70342d11b17754ac95d00d6b71df0cd632 Mon Sep 17 00:00:00 2001
From: Ivan Kohler <ivan@freeside.biz>
Date: Sun, 14 Jun 2015 14:26:45 -0700
Subject: why wouldn't we want to hide the whole thing rather than leave a big
 blank gap?

---
 httemplate/edit/cust_main.cgi | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'httemplate/edit/cust_main.cgi')

diff --git a/httemplate/edit/cust_main.cgi b/httemplate/edit/cust_main.cgi
index d4129a37c..d203e0044 100755
--- a/httemplate/edit/cust_main.cgi
+++ b/httemplate/edit/cust_main.cgi
@@ -97,12 +97,10 @@
 
 <SCRIPT>
 function samechanged(what) {
-%#  document.getElementById('table_ship_location').style.visibility = 
   var t1 = document.getElementById('table_ship_location');
   if ( what.checked ) {
     t1.style.display = 'none';
-  }
-  else {
+  } else {
     t1.style.display = ''
   }
 }
-- 
cgit v1.2.1


From 5b507bd1cb845ef44a49f84d4f0ae20ec80118d5 Mon Sep 17 00:00:00 2001
From: Ivan Kohler <ivan@freeside.biz>
Date: Sun, 14 Jun 2015 14:56:51 -0700
Subject: UI spring cleaning: customer edit

---
 httemplate/edit/cust_main.cgi | 2 --
 1 file changed, 2 deletions(-)

(limited to 'httemplate/edit/cust_main.cgi')

diff --git a/httemplate/edit/cust_main.cgi b/httemplate/edit/cust_main.cgi
index d203e0044..f0de6e14a 100755
--- a/httemplate/edit/cust_main.cgi
+++ b/httemplate/edit/cust_main.cgi
@@ -107,8 +107,6 @@ function samechanged(what) {
 samechanged(document.getElementById('same'));
 </SCRIPT>
 
-<BR>
-
 <& cust_main/contacts_new.html, 'cust_main'=>$cust_main, &>
 
 %# billing info
-- 
cgit v1.2.1


From 925cf9600781e534b4188adb68cc53a9709a6569 Mon Sep 17 00:00:00 2001
From: Ivan Kohler <ivan@freeside.biz>
Date: Tue, 16 Jun 2015 16:49:27 -0700
Subject: UI spring cleaning: include jquery in all pages, animate customer
 edit service address hide/unhide

---
 httemplate/edit/cust_main.cgi | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

(limited to 'httemplate/edit/cust_main.cgi')

diff --git a/httemplate/edit/cust_main.cgi b/httemplate/edit/cust_main.cgi
index f0de6e14a..effe84b96 100755
--- a/httemplate/edit/cust_main.cgi
+++ b/httemplate/edit/cust_main.cgi
@@ -68,8 +68,8 @@
            VALUE="Y"
            <% $has_ship_address ? '' : 'CHECKED' %>
     ><% mt('same as billing address') |h %>
-    <DIV CLASS="fsinnerbox">
-      <TABLE ID="table_ship_location" WIDTH="100%">
+    <DIV ID="div_ship_location">
+      <TABLE WIDTH="100%" CLASS="fsinnerbox">
       <& cust_main/before_ship_location.html, $cust_main &>
       <& /elements/location.html,
           object => $cust_main->ship_location,
@@ -95,16 +95,20 @@
   </TD>
 </TR></TABLE>
 
-<SCRIPT>
+<SCRIPT TYPE="text/javascript">
+
 function samechanged(what) {
-  var t1 = document.getElementById('table_ship_location');
   if ( what.checked ) {
-    t1.style.display = 'none';
+    $('#div_ship_location').slideUp();
   } else {
-    t1.style.display = ''
+    $('#div_ship_location').slideDown();
   }
 }
-samechanged(document.getElementById('same'));
+
+% if ( ! $has_ship_address ) {
+  $('#div_ship_location').hide();
+% }
+
 </SCRIPT>
 
 <& cust_main/contacts_new.html, 'cust_main'=>$cust_main, &>
-- 
cgit v1.2.1