From 4a171d1b07ad533ceb44a0cef4193b2f9ec315ad Mon Sep 17 00:00:00 2001
From: Dan Scott <dscott@laurentian.ca>
Date: Mon, 15 Jul 2013 14:36:30 -0400
Subject: [PATCH] Improved copy table CSS/markup for conjoined bibs

Let the library name column wrap now that it may potentially contain
stupidly long title/author combos. Also, associate the table cells with
their corresponding headers for improved accessibility.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
---
 Open-ILS/src/templates/opac/css/style.css.tt2      |  9 +++++++--
 .../src/templates/opac/parts/record/copy_table.tt2 | 14 +++++++-------
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 4490089d9f..39cd573f3d 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -448,9 +448,14 @@ div.format_icon {
     padding-top: 1.5em;
 }
 
-#rdetails_status td, #rdetails_status2 td {
-    white-space:nowrap !important;
+#rdetails_status td {
     padding: 7px 0px 3px 13px;
+    white-space: nowrap;
+}
+
+#rdetails_status td[header=copy_header_library], 
+#rdetails_status td[header=copy_header_shelfloc] {
+    white-space: normal;
 }
 
 #rdetails_status thead th {
diff --git a/Open-ILS/src/templates/opac/parts/record/copy_table.tt2 b/Open-ILS/src/templates/opac/parts/record/copy_table.tt2
index d558c85402..5bab2b7890 100644
--- a/Open-ILS/src/templates/opac/parts/record/copy_table.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/copy_table.tt2
@@ -54,7 +54,7 @@ END;
     <tbody class="copy_details_table">
 [%- FOREACH peer IN ctx.foreign_copies;
         FOREACH bib IN peer.foreign_copy_maps; %]
-    <tr><td>
+    <tr><td header='copy_header_shelfloc'>
     [%- bib_lib_name = ctx.get_aou(bib.target_copy.circ_lib).name | html;
         l("[_1] (foreign item)", bib_lib_name); -%]
         <ul><li>
@@ -65,11 +65,11 @@ END;
             </a>
         </li></ul>
     </td>
-    <td>[% bib.target_copy.call_number.label | html %]</td>
-    <td>[% bib.target_copy.barcode | html %]</td>
-    <td>[% copy_info.copy_location | html %]</td>
-    <td>[% copy_info.copy_status | html %]</td>
-    <td>[% copy_info.due_date | html %]</td>
+    <td header='copy_header_callnumber'>[% bib.target_copy.call_number.label | html %]</td>
+    <td header='copy_header_barcode'>[% bib.target_copy.barcode | html %]</td>
+    <td header='copy_header_shelfloc'>[% copy_info.copy_location | html %]</td>
+    <td header='copy_header_status'>[% copy_info.copy_status | html %]</td>
+    <td header='copy_header_due_date'>[% copy_info.due_date | html %]</td>
 </tr>
    [%- END; # FOREACH peer
 END; # FOREACH bib
@@ -189,7 +189,7 @@ END; # FOREACH bib
             [% END %]
         [% END %]
 
-<tr><td>
+<tr><td header='copy_header_shelfloc'>
 [%- IF copy_info.peer_bib_marc.size > 1;
 '<ul>';
     FOREACH bib IN copy_info.peer_bib_marc;
-- 
2.43.2