[freeside-commits] branch master updated. 1cde870268b03197da75f089669d60a7ba3c5617

Ivan ivan at 420.am
Sat May 16 16:09:00 PDT 2015


The branch, master has been updated
       via  1cde870268b03197da75f089669d60a7ba3c5617 (commit)
       via  747949f05c140e23221b3a2364928431f58fc2d3 (commit)
      from  16fe3ab085213efe13680a7d2d8c50e38cf8750a (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 1cde870268b03197da75f089669d60a7ba3c5617
Merge: 747949f 16fe3ab
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Sat May 16 16:08:31 2015 -0700

    Merge branch 'master' of git.freeside.biz:/home/git/freeside


commit 747949f05c140e23221b3a2364928431f58fc2d3
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Sat May 16 16:08:15 2015 -0700

    customer view UI

diff --git a/httemplate/elements/freeside.css b/httemplate/elements/freeside.css
index d038f6f..280427c 100644
--- a/httemplate/elements/freeside.css
+++ b/httemplate/elements/freeside.css
@@ -225,18 +225,30 @@ div.fstabcontainer {
 }
 
 .fsinnerbox {
-  background-color:#cccccc;
+  /* background-color:#cccccc; */
   padding:2px;
-  -moz-box-shadow:  1px 1px 2px #666666;
+  border: 1px solid #dddddd;
+  /* -moz-box-shadow:  1px 1px 2px #666666;
   -webkit-box-shadow:  1px 1px 2px #666666;
   box-shadow: 1px 1px 2px #666666;
   filter: progid:DXImageTransform.Microsoft.Shadow(color='#666666', Direction=135, Strength=2);
+  */
+}
+
+.fsinnerbox th {
+  font-weight:normal;
+  font-size:75%;
+  valign: bottom;
+  color: #999999;
 }
 
 .fsinnerbox-title {
   font-size:110%;
   font-weight:bold;
-  background-color:#cccccc;
+  /*border-top: 1px solid #dddddd;
+  border-left: 1px solid #dddddd;
+  border-right: 1px solid #dddddd; */
+  /* background-color:#cccccc;
   padding:2px;
          -moz-border-radius-topleft:8px;
          -moz-border-radius-topright:8px;
@@ -248,6 +260,7 @@ div.fstabcontainer {
   -webkit-box-shadow:  1px 0px 1px #999999;
   box-shadow: 1px 0px 1px #999999;
   filter: progid:DXImageTransform.Microsoft.Shadow(color='#999999', Direction=90, Strength=1);
+  */
 }
 
 .background {
diff --git a/httemplate/elements/tr-coords.html b/httemplate/elements/tr-coords.html
index 5ac5ed4..ef5e76b 100644
--- a/httemplate/elements/tr-coords.html
+++ b/httemplate/elements/tr-coords.html
@@ -1,9 +1,7 @@
 <TR>
-  <TD ALIGN="right"><% mt('Latitude') |h %></TD>
+  <TH ALIGN="right"><% mt('Coordinates') |h %></TH>
   <TD COLSPAN=7>
-    <FONT STYLE="background-color: #ffffff; border: 1px solid #ffffff"><% $latitude %></FONT>
-     <% mt('Longitude') |h %>
-    <FONT STYLE="background-color: #ffffff; border: 1px solid #ffffff"><% $longitude %></FONT>
+    <% $latitude %>, <% $longitude %>
     <& /elements/coord-links.html, @_ &>
   </TD>
 </TR>
diff --git a/httemplate/view/cust_main/billing.html b/httemplate/view/cust_main/billing.html
index 4f4b745..84241d2 100644
--- a/httemplate/view/cust_main/billing.html
+++ b/httemplate/view/cust_main/billing.html
@@ -20,8 +20,8 @@
 
 % if ( $cust_main->complimentary ) {
   <TR>
-    <TD ALIGN="right"><% mt('Complimentary') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $yes %></TD>
+    <TH ALIGN="right"><% mt('Complimentary') |h %></TH>
+    <TD><% $yes %></TD>
   </TR>
 % }
 
@@ -29,14 +29,14 @@
 %  =~ s/^(\-?)(.*)$/<FONT SIZE=+1>$1<\/FONT>$money_char$2/;
 
 <TR>
-  <TD ALIGN="right"><% mt('Balance due') |h %></TD>
-  <TD BGCOLOR="#ffffff"><B><% $balance %></B></TD>
+  <TH ALIGN="right"><% mt('Balance due') |h %></TH>
+  <TD><B><% $balance %></B></TD>
 </TR>
 
 % if ( $conf->exists('cust_main-select-prorate_day') ) {
 <TR>
-  <TD ALIGN="right"><% mt('Prorate day of month') |h %></TD>
-  <TD BGCOLOR="#ffffff"><% $cust_main->prorate_day %>
+  <TH ALIGN="right"><% mt('Prorate day of month') |h %></TH>
+  <TD><% $cust_main->prorate_day %>
   </TD>
 </TR>
 % }
@@ -50,16 +50,16 @@
 %    )
 % {
     <TR>
-      <TD ALIGN="right"><% mt('Payment day of month') |h %></TD>
-      <TD BGCOLOR="#ffffff"><% $cust_main->billday %>
+      <TH ALIGN="right"><% mt('Payment day of month') |h %></TH>
+      <TD><% $cust_main->billday %>
       </TD>
     </TR>
 % }
 
 % if ( $cust_main->po_number ) { 
     <TR>
-      <TD ALIGN="right"><% mt('Purchase Order #') |h %></TD>
-      <TD BGCOLOR="#ffffff"><% $cust_main->po_number %></TD>
+      <TH ALIGN="right"><% mt('Purchase Order #') |h %></TH>
+      <TD><% $cust_main->po_number %></TD>
     </TR>
 % } 
 
@@ -70,16 +70,16 @@
 %        )
 % {
     <TR>
-      <TD ALIGN="right"><% mt('Tax exempt') |h %><% @exempt_groups ? ' ('.emt('all taxes').')' : '' %></TD>
-      <TD BGCOLOR="#ffffff"><% $cust_main->tax ? $yes : $no %></TD>
+      <TH ALIGN="right"><% mt('Tax exempt') |h %><% @exempt_groups ? ' ('.emt('all taxes').')' : '' %></TH>
+      <TD><% $cust_main->tax ? $yes : $no %></TD>
     </TR>
 % }
 
 % foreach my $exempt_group ( @exempt_groups ) {
 %   my $cust_main_exemption = $cust_main->tax_exemption($exempt_group);
     <TR>
-      <TD ALIGN="right"><% mt('Tax exempt') |h %> (<% $exempt_group %> taxes)</TD>
-      <TD BGCOLOR="#ffffff"><% $cust_main_exemption ? $yes : $no %>
+      <TH ALIGN="right"><% mt('Tax exempt') |h %> (<% $exempt_group %> taxes)</TH>
+      <TD><% $cust_main_exemption ? $yes : $no %>
         <% $cust_main_exemption ? $cust_main_exemption->exempt_number : '' |h %>
       </TD>
     </TR>
@@ -87,16 +87,16 @@
 
 % if ( $conf->exists('enable_taxproducts') ) {
 <TR>
-  <TD ALIGN="right"><% mt('Tax location') |h %></TD>
+  <TH ALIGN="right"><% mt('Tax location') |h %></TH>
 % my $tax_location = $conf->exists('tax-ship_address')
 %                    ? $cust_main->ship_location
 %                    : $cust_main->bill_location;
-  <TD BGCOLOR="#ffffff"><% $tax_location->geocode('cch') %></TD>
+  <TD><% $tax_location->geocode('cch') %></TD>
 </TR>
 % }
 <TR>
-  <TD ALIGN="right"><% mt('Postal mail invoices') |h %></TD>
-  <TD BGCOLOR="#ffffff">
+  <TH ALIGN="right"><% mt('Postal mail invoices') |h %></TH>
+  <TD>
     <% ( grep { $_ eq 'POST' } @invoicing_list )
          ? $yes. ( $cust_main->invoice_attn
                      ? ', attn: '. $cust_main->invoice_attn
@@ -107,15 +107,15 @@
   </TD>
 </TR>
 <TR>
-  <TD ALIGN="right"><% mt('Email invoices') |h %></TD>
-  <TD BGCOLOR="#ffffff">
+  <TH ALIGN="right"><% mt('Email invoices') |h %></TH>
+  <TD>
     <% $cust_main->invoice_noemail ? $no : $yes %>
   </TD>
 </TR>
 % unless ( $conf->exists('cust-email-high-visibility')) {
 <TR>
-  <TD ALIGN="right"><% mt('Email address(es)') |h %></TD>
-  <TD BGCOLOR="#ffffff">
+  <TH ALIGN="right"><% mt('Email address(es)') |h %></TH>
+  <TD>
     <% join(', ', grep { $_ !~ /^(POST|FAX)$/ } @invoicing_list ) || $no %>
 %   if ( $cust_main->message_noemail ) {
     <BR>
@@ -125,8 +125,8 @@
 </TR>
 % }
 <TR>
-  <TD ALIGN="right"><% mt('Invoice terms') |h %></TD>
-  <TD BGCOLOR="#ffffff">
+  <TH ALIGN="right"><% mt('Invoice terms') |h %></TH>
+  <TD>
     <% $cust_main->invoice_terms
          || emt('Default').' ('. ( $conf->config('invoice_default_terms', $cust_main->agentnum)
                                      || emt('Payable upon receipt')
@@ -136,8 +136,8 @@
   </TD>
 </TR>
 <TR>
-  <TD ALIGN="right"><% mt('Credit limit') |h %></TD>
-  <TD BGCOLOR="#ffffff">
+  <TH ALIGN="right"><% mt('Credit limit') |h %></TH>
+  <TD>
 %   my $default_credit_limit = $conf->config('default_credit_limit');
     <% length($cust_main->credit_limit)
          ? $money_char. sprintf("%.2f", $cust_main->credit_limit)
@@ -153,51 +153,51 @@
 
 % if ( $conf->exists('voip-cust_cdr_spools') ) { 
   <TR>
-    <TD ALIGN="right"><% mt('Spool CDRs') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->spool_cdr ? $yes : $no %></TD>
+    <TH ALIGN="right"><% mt('Spool CDRs') |h %></TH>
+    <TD><% $cust_main->spool_cdr ? $yes : $no %></TD>
   </TR>
 % } 
 
 % if ( $conf->exists('voip-cust_cdr_squelch') ) { 
   <TR>
-    <TD ALIGN="right"><% mt($conf->exists('voip-cdr_email') ? 'Print CDRs' : 'Show CDRs') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->squelch_cdr ? $no : $yes %></TD>
+    <TH ALIGN="right"><% mt($conf->exists('voip-cdr_email') ? 'Print CDRs' : 'Show CDRs') |h %></TH>
+    <TD><% $cust_main->squelch_cdr ? $no : $yes %></TD>
   </TR>
 % } 
 
 % if ( $conf->exists('voip-cust_accountcode_cdr') ) { 
    <TR>
-     <TD ALIGN="right"><% mt('Breakdown CDRs by accountcode') |h %></TD>
-     <TD BGCOLOR="#ffffff"><% $cust_main->accountcode_cdr ? $yes : $no %></TD>
+     <TH ALIGN="right"><% mt('Breakdown CDRs by accountcode') |h %></TH>
+     <TD><% $cust_main->accountcode_cdr ? $yes : $no %></TD>
    </TR>
 % }
 
 % if ( $conf->exists('voip-cust_email_csv_cdr') ) { 
   <TR>
-    <TD ALIGN="right"><% mt('Email CDRs as CSV') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->email_csv_cdr ? $yes : $no %></TD>
+    <TH ALIGN="right"><% mt('Email CDRs as CSV') |h %></TH>
+    <TD><% $cust_main->email_csv_cdr ? $yes : $no %></TD>
   </TR>
 % } 
 
 % if ( $show_term || $cust_main->cdr_termination_percentage ) {
   <TR>
-    <TD ALIGN="right"><% mt('CDR termination settlement') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->cdr_termination_percentage %><% $cust_main->cdr_termination_percentage =~ /\d/ ? '%' : '' %></TD>
+    <TH ALIGN="right"><% mt('CDR termination settlement') |h %></TH>
+    <TD><% $cust_main->cdr_termination_percentage %><% $cust_main->cdr_termination_percentage =~ /\d/ ? '%' : '' %></TD>
   </TR>
 % }
 
 % if ( $cust_main->currency ) {
   <TR>
-    <TD ALIGN="right"><% mt('Invoicing currency') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->currency. ': '. code2currency($cust_main->currency) %></TD>
+    <TH ALIGN="right"><% mt('Invoicing currency') |h %></TH>
+    <TD><% $cust_main->currency. ': '. code2currency($cust_main->currency) %></TD>
   </TR>
 % }
 
 % if ( $cust_main->locale ) {
 % my %locale_info = FS::Locales->locale_info($cust_main->locale);
   <TR>
-    <TD ALIGN="right"><% mt('Invoicing locale') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $locale_info{name} . " (" . $locale_info{country} .")" %></TD>
+    <TH ALIGN="right"><% mt('Invoicing locale') |h %></TH>
+    <TD><% $locale_info{name} . " (" . $locale_info{country} .")" %></TD>
   </TR>
 % }
 
diff --git a/httemplate/view/cust_main/contacts.html b/httemplate/view/cust_main/contacts.html
index a5bafea..83a6380 100644
--- a/httemplate/view/cust_main/contacts.html
+++ b/httemplate/view/cust_main/contacts.html
@@ -25,11 +25,11 @@
 % if ( $this eq 'bill' ) {
 %   #billing contact fields
   <TR>
-    <TD ALIGN="right"><% mt('Contact name') |h %></TD>
-    <TD COLSPAN=5 BGCOLOR="#ffffff"><% $cust_main->contact |h %></TD>
+    <TH ALIGN="right"><% mt('Contact name') |h %></TH>
+    <TD COLSPAN=5><% $cust_main->contact |h %></TD>
 %   if ( $conf->exists('show_ss') ) {
-    <TD ALIGN="right"><% mt('SS#') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $conf->exists('unmask_ss')
+    <TH ALIGN="right"><% mt('SS#') |h %></TH>
+    <TD><% $conf->exists('unmask_ss')
                               ? $cust_main->ss
                               : $cust_main->masked('ss') || ' ' %></TD>
 %   }
@@ -37,8 +37,8 @@
 %   if ( $conf->exists('cust_main-enable_spouse') and
 %        ($cust_main->spouse_last or $cust_main->spouse_first) ) {
   <TR>
-    <TD ALIGN="right"><% mt('Spouse') |h %></TD>
-    <TD COLSPAN=5 BGCOLOR="#ffffff">
+    <TH ALIGN="right"><% mt('Spouse') |h %></TH>
+    <TD COLSPAN=5>
       <% join(', ', grep $_, 
                     $cust_main->spouse_last, $cust_main->spouse_first) %>
     </TD>
@@ -46,7 +46,7 @@
 % }
 %   if ( $conf->exists('cust-email-high-visibility') ) {
   <TR>
-    <TD ALIGN="right"><% mt('Email address(es)') |h %></TD>
+    <TH ALIGN="right"><% mt('Email address(es)') |h %></TH>
     <TD BGCOLOR="#ffff00">
       <% $cust_main->invoicing_list_emailonly_scalar || $no %>
     </TD>
@@ -54,15 +54,15 @@
 %   }
 %   if ( $cust_main->company ) {
   <TR>
-    <TD ALIGN="right"><% mt('Company') |h %></TD>
-    <TD COLSPAN=7 BGCOLOR="#ffffff"><% $cust_main->company |h %></TD>
+    <TH ALIGN="right"><% mt('Company') |h %></TH>
+    <TD COLSPAN=7><% $cust_main->company |h %></TD>
   </TR>
 %   }
 % } elsif ( $this eq 'ship' ) {
 %   if ( $cust_main->ship_company ) { # mostly obsolete these days...
   <TR>
     <TD ALIGN="right"><% mt('Company') |h %></TD>
-    <TD COLSPAN=7 BGCOLOR="#ffffff"><% $cust_main->ship_company |h %></TD>
+    <TD COLSPAN=7><% $cust_main->ship_company |h %></TD>
   </TR>
 %   }
 % }
@@ -73,13 +73,13 @@
 %   if ( $location->locationname ) {
       <TR>
         <TD ALIGN="right"><% mt('Location ID') |h %></TD>
-        <TD COLSPAN=7 BGCOLOR="#ffffff"><% $location->locationname |h %></TD>
+        <TD COLSPAN=7><% $location->locationname |h %></TD>
       </TR>
 %   }
 
 <TR>
-  <TD ALIGN="right"><% mt('Address') |h %></TD>
-  <TD COLSPAN=7 BGCOLOR="#ffffff"><% $location->address1 |h %></TD>
+  <TH ALIGN="right"><% mt('Address') |h %></TH>
+  <TD COLSPAN=7><% $location->address1 |h %></TD>
 </TR>
 
 % if ( $location->get('address2') ) {
@@ -88,27 +88,25 @@
 %                        : ' ';
 
 <TR>
-  <TD ALIGN="right"><% $address2_label %></TD>
-  <TD COLSPAN=7 BGCOLOR="#ffffff"><% $location->address2 |h %></TD>
+  <TH ALIGN="right"><% $address2_label %></TH>
+  <TD COLSPAN=7><% $location->address2 |h %></TD>
 </TR>
 
 % } 
 
 <TR>
-  <TD ALIGN="right"><% mt('City') |h %></TD>
-  <TD BGCOLOR="#ffffff"><% $location->city |h %></TD>
+  <TH></TH>
+  <TD COLSPAN=5><% $location->city |h %>\
 % if ( $location->county ) {
-    <TD ALIGN="right"><% mt('County') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $location->county |h %></TD>
+ (<% $location->county |h %> county)\
 % }
-  <TD ALIGN="right"><% mt('State') |h %></TD>
-  <TD BGCOLOR="#ffffff"><% state_label( $location->state, $location->country ) |h %></TD>
-  <TD ALIGN="right"><% mt('Zip') |h %></TD>
-  <TD BGCOLOR="#ffffff"><% $location->zip %></TD>
+,<% state_label( $location->state, $location->country ) |h %>
+  <% $location->zip %>
+</TD>
 </TR>
 <TR>
-  <TD ALIGN="right"><% mt('Country') |h %></TD>
-  <TD BGCOLOR="#ffffff"><% code2country( $location->country ) %></TD>
+  <TH></TH>
+  <TD><% code2country( $location->country ) %></TD>
 </TR>
 
 % if ( $location->latitude && $location->longitude ) {
@@ -128,7 +126,7 @@
 %     next if !$cust_main->get($phone);
 <TR>
   <TD ALIGN="right"><% $phone_label{$phone} %></TD>
-  <TD COLSPAN=3 BGCOLOR="#ffffff">
+  <TD COLSPAN=3>
     <& /elements/phonenumber.html,
         $cust_main->get($phone),
         callable => 1,
@@ -142,7 +140,7 @@
 
   <TR>
     <TD ALIGN="right"><% mt('Fax') |h %></TD>
-    <TD COLSPAN=3 BGCOLOR="#ffffff">
+    <TD COLSPAN=3>
       <% $cust_main->get('fax') || ' ' %>
     </TD>
   </TR>
@@ -153,9 +151,9 @@
 
 <TR>
     <TD ALIGN="right"><% $stateid_label %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->masked('stateid') || '&nbsp' %></TD>
+    <TD><% $cust_main->masked('stateid') || '&nbsp' %></TD>
     <TD ALIGN="right"><% $stateid_state_label %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->stateid_state || '&nbsp' %></TD>
+    <TD><% $cust_main->stateid_state || '&nbsp' %></TD>
   </TR>
 
 %   }
diff --git a/httemplate/view/cust_main/cust_payby.html b/httemplate/view/cust_main/cust_payby.html
index 90ce96e..a91c425 100644
--- a/httemplate/view/cust_main/cust_payby.html
+++ b/httemplate/view/cust_main/cust_payby.html
@@ -13,36 +13,36 @@
 
 %       my $auto = $cust_payby->payby eq 'CARD' ? 'automatic' : 'on-demand';
         <TR>
-          <TD COLSPAN=2 ALIGN="center" BGCOLOR="#ffffff">
+          <TD COLSPAN=2 ALIGN="center">
             <% mt("Credit card ([_1])",$auto) |h %>
           </TD>
         </TR>
         <TR>
-          <TD ALIGN="right"><% mt('Card number') |h %></TD>
-          <TD BGCOLOR="#ffffff"><% $cust_payby->paymask %></TD>
+          <TH ALIGN="right"><% mt('Card number') |h %></TH>
+          <TD><% $cust_payby->paymask %></TD>
         </TR>
 
 %       my( $mon, $year ) = $cust_payby->paydate_mon_year;
         <TR>
-          <TD ALIGN="right"><% mt('Expiration') |h %></TD>
-          <TD BGCOLOR="#ffffff"><% "$mon/$year" %></TD>
+          <TH ALIGN="right"><% mt('Expiration') |h %></TH>
+          <TD><% "$mon/$year" %></TD>
         </TR>
 
 %       if ( $cust_payby->paystart_month ) { 
           <TR>
-            <TD ALIGN="right"><% mt('Start date') |h %></TD>
-            <TD BGCOLOR="#ffffff"><% $cust_payby->paystart_month. '/'. $cust_payby->paystart_year %>
+            <TH ALIGN="right"><% mt('Start date') |h %></TH>
+            <TD><% $cust_payby->paystart_month. '/'. $cust_payby->paystart_year %>
           </TR>
 %       } elsif ( $cust_payby->payissue ) { 
           <TR>
-            <TD ALIGN="right"><% mt('Issue #') |h %></TD>
-            <TD BGCOLOR="#ffffff"><% $cust_payby->payissue %>
+            <TH ALIGN="right"><% mt('Issue #') |h %></TH>
+            <TD><% $cust_payby->payissue %>
           </TR>
 %       } 
 
         <TR>
-          <TD ALIGN="right"><% mt('Name on card') |h %></TD>
-          <TD BGCOLOR="#ffffff"><% $cust_payby->payname %></TD>
+          <TH ALIGN="right"><% mt('Name on card') |h %></TH>
+          <TD><% $cust_payby->payname %></TD>
         </TR>
 
 %     } elsif ( $cust_payby->payby eq 'CHEK' || $cust_payby->payby eq 'DCHK') {
@@ -55,7 +55,7 @@
 %         if $conf->config('echeck-country') eq 'CA';
 
         <TR>
-          <TD COLSPAN=2 ALIGN="center" BGCOLOR="#ffffff">
+          <TD COLSPAN=2 ALIGN="center">
             <% mt("Electronic check ([_1])",$auto) |h %>
           </TD>
         </TR>
@@ -65,34 +65,34 @@
 %                             ? 'ABA/Routing number'
 %                             : 'Routing number';
         <TR>
-          <TD ALIGN="right"><% mt($routing_label) |h %></TD>
-          <TD BGCOLOR="#ffffff"><% $aba %></TD>
+          <TH ALIGN="right"><% mt($routing_label) |h %></TH>
+          <TD><% $aba %></TD>
         </TR>
 
 %       if ( $conf->config('echeck-country') eq 'CA' ) {
           <TR>
-            <TD ALIGN="right"><% mt('Branch number') |h %></TD>
-            <TD BGCOLOR="#ffffff"><% $branch %></TD>
+            <TH ALIGN="right"><% mt('Branch number') |h %></TH>
+            <TD><% $branch %></TD>
           </TR>
 %       }
 
         <TR>
-          <TD ALIGN="right"><% mt('Account number') |h %></TD>
-          <TD BGCOLOR="#ffffff"><% $account %></TD>
+          <TH ALIGN="right"><% mt('Account number') |h %></TH>
+          <TD><% $account %></TD>
         </TR>
         <TR>
-          <TD ALIGN="right"><% mt('Account type') |h %></TD>
-          <TD BGCOLOR="#ffffff"><% $cust_payby->paytype %></TD>
+          <TH ALIGN="right"><% mt('Account type') |h %></TH>
+          <TD><% $cust_payby->paytype %></TD>
         </TR>
         <TR>
-          <TD ALIGN="right"><% mt('Bank name') |h %></TD>
-          <TD BGCOLOR="#ffffff"><% $cust_payby->payname %></TD>
+          <TH ALIGN="right"><% mt('Bank name') |h %></TH>
+          <TD><% $cust_payby->payname %></TD>
         </TR>
 
 %       if ( $conf->exists('show_bankstate') ) {
           <TR>
-            <TD ALIGN="right"><% $paystate_label %></TD>
-            <TD BGCOLOR="#ffffff"><% $cust_payby->paystate || '   ' %></TD>
+            <TH ALIGN="right"><% $paystate_label %></TH>
+            <TD><% $cust_payby->paystate || '   ' %></TD>
           </TR>
 %       }
 
diff --git a/httemplate/view/cust_main/misc.html b/httemplate/view/cust_main/misc.html
index f0ecf8f..7e625d2 100644
--- a/httemplate/view/cust_main/misc.html
+++ b/httemplate/view/cust_main/misc.html
@@ -29,15 +29,15 @@
 
 %  #if ( $cust_main->classnum ) {
      <TR>
-       <TD ALIGN="right"><% mt('Class') |h %></TD>
-       <TD BGCOLOR="#ffffff"><% $cust_main->classname || '('.emt('none').')' %></TD>
+       <TH ALIGN="right"><% mt('Class') |h %></TH>
+       <TD><% $cust_main->classname || '('.emt('none').')' %></TD>
      </TR>
 %  #}
 
 % if ( $cust_main->salesnum ) {
     <TR>
-      <TD ALIGN="right"><% mt('Sales Person') |h %></TD>
-      <TD BGCOLOR="#ffffff"><% $cust_main->salesperson |h %></TD>
+      <TH ALIGN="right"><% mt('Sales Person') |h %></TH>
+      <TD><% $cust_main->salesperson |h %></TD>
     </TR>
 % }
 
@@ -47,15 +47,15 @@
 %    } );
 
 <TR>
-  <TD ALIGN="right"><% mt('Advertising source') |h %></TD>
-  <TD BGCOLOR="#ffffff"><% $referral->refnum %>: <% $referral->referral%></TD>
+  <TH ALIGN="right"><% mt('Advertising source') |h %></TH>
+  <TD><% $referral->refnum %>: <% $referral->referral%></TD>
 </TR>
 % } 
 
 
 <TR>
-  <TD ALIGN="right"><% mt('Referring Customer') |h %></TD>
-  <TD BGCOLOR="#ffffff">
+  <TH ALIGN="right"><% mt('Referring Customer') |h %></TH>
+  <TD>
 %
 %  my $referring_cust_main = '';
 %  if ( $cust_main->referral_custnum
@@ -82,13 +82,13 @@
 </TR>
 
 <TR>
-  <TD ALIGN="right"><% mt('Order taker') |h %></TD>
-  <TD BGCOLOR="#ffffff"><% $cust_main->otaker %></TD>
+  <TH ALIGN="right"><% mt('Order taker') |h %></TH>
+  <TD><% $cust_main->otaker %></TD>
 </TR>
 
   <TR>
-    <TD ALIGN="right"><% mt('Signup Date') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->signupdate ? time2str($date_format, $cust_main->signupdate) : '' %></TD>
+    <TH ALIGN="right"><% mt('Signup Date') |h %></TH>
+    <TD><% $cust_main->signupdate ? time2str($date_format, $cust_main->signupdate) : '' %></TD>
   </TR>
 
 % my $id_country = $conf->config('national_id-country');
@@ -97,14 +97,14 @@
       <TR>
 %     my($old, $nric) = ( '', '');
 %     if ( $cust_main->national_id =~ /^\d{6}\-\d{2}\-\d{4}$/ ) {
-          <TD ALIGN="right"><% mt('NRIC') |h %></TD>
+          <TH ALIGN="right"><% mt('NRIC') |h %></TH>
 %     } else { # elsif ( $cust_main->national_id =~ /^\w\d{9}$/ ) {
-          <TD ALIGN="right"><% mt('Old IC/Passport') |h %></TD>
+          <TH ALIGN="right"><% mt('Old IC/Passport') |h %></TH>
 %     #} else {
 %     #  warn "unknown national_id format";
-%#         <TD ALIGN="right"></TD>
+%#         <TH ALIGN="right"></TH>
 %     }
-        <TD BGCOLOR="#ffffff"><% $cust_main->national_id |h %></TD>
+        <TD><% $cust_main->national_id |h %></TD>
       </TR>
 %   } else {
 %     warn "unknown national_id-country $id_country";
@@ -119,8 +119,8 @@
 %              : '';
 
   <TR>
-    <TD ALIGN="right"><% mt('Date of Birth') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $dt ? $dt->strftime($date_format) : '' %></TD>
+    <TH ALIGN="right"><% mt('Date of Birth') |h %></TH>
+    <TD><% $dt ? $dt->strftime($date_format) : '' %></TD>
   </TR>
 
 %   if ( $conf->exists('cust_main-enable_spouse') 
@@ -131,8 +131,8 @@
 %              );
 
   <TR>
-    <TD ALIGN="right"><% mt('Spouse Date of Birth') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $dt ? $dt->strftime($date_format) : '' %></TD>
+    <TH ALIGN="right"><% mt('Spouse Date of Birth') |h %></TH>
+    <TD><% $dt ? $dt->strftime($date_format) : '' %></TD>
   </TR>
 
 %   }
@@ -146,8 +146,8 @@
 %              : '';
 
   <TR>
-    <TD ALIGN="right"><% mt('Anniversary Date') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $dt ? $dt->strftime($date_format) : '' %></TD>
+    <TH ALIGN="right"><% mt('Anniversary Date') |h %></TH>
+    <TD><% $dt ? $dt->strftime($date_format) : '' %></TD>
   </TR>
 
 % }
@@ -155,8 +155,8 @@
 % if ( $cust_main->district ) {
 
   <TR>
-    <TD ALIGN="right"><% mt('Tax district') |h %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->ship_location->district %></TD>
+    <TH ALIGN="right"><% mt('Tax district') |h %></TH>
+    <TD><% $cust_main->ship_location->district %></TD>
   </TR>
 
 % }
@@ -165,8 +165,8 @@
 % if ( $conf->exists('ticket_system-selfservice_edit_subject') ) {
 
   <TR>
-    <TD ALIGN="right"><% mt('Edit ticket subjects') %></TD>
-    <TD BGCOLOR="#ffffff"><% $cust_main->edit_subject ? 'yes' : 'no' %></TD>
+    <TH ALIGN="right"><% mt('Edit ticket subjects') %></TH>
+    <TD><% $cust_main->edit_subject ? 'yes' : 'no' %></TD>
   </TR>
 
 % }

-----------------------------------------------------------------------

Summary of changes:
 httemplate/elements/freeside.css          |   19 +++++--
 httemplate/elements/tr-coords.html        |    6 +--
 httemplate/view/cust_main/billing.html    |   80 ++++++++++++++---------------
 httemplate/view/cust_main/contacts.html   |   56 ++++++++++----------
 httemplate/view/cust_main/cust_payby.html |   48 ++++++++---------
 httemplate/view/cust_main/misc.html       |   52 +++++++++----------
 6 files changed, 135 insertions(+), 126 deletions(-)




More information about the freeside-commits mailing list