From 8b1f9ca5d3aa6a2537b128bb1ffe72824ed7a7bb Mon Sep 17 00:00:00 2001
From: William Rockwood <wrockwood@tadl.org>
Date: Tue, 19 Mar 2013 14:09:44 -0400
Subject: [PATCH] Padding and spacing changes on holds/hold_history

- Re-imagineered the holds and hold history display
- Padding, alignment, spacing.
- Needs more work. A lot more work.

Signed-off-by: William Rockwood <wrockwood@tadl.org>
---
 css/tadl.css                                    |  1 +
 templates_tadlskin/opac/myopac/hold_history.tt2 |  4 +--
 templates_tadlskin/opac/myopac/holds.tt2        | 36 ++++++++++++++++---------
 3 files changed, 26 insertions(+), 15 deletions(-)

diff --git a/css/tadl.css b/css/tadl.css
index 9a1fca8458..ea2a6120f8 100644
--- a/css/tadl.css
+++ b/css/tadl.css
@@ -1086,6 +1086,7 @@ div.result_place_hold {
 	text-transform:uppercase;
 	font-size: 10px;
 	padding-right:10px;
+	width:940px;
 }
 
 #acct_checked_main_header td,
diff --git a/templates_tadlskin/opac/myopac/hold_history.tt2 b/templates_tadlskin/opac/myopac/hold_history.tt2
index c781853f9f..d5114af4ce 100644
--- a/templates_tadlskin/opac/myopac/hold_history.tt2
+++ b/templates_tadlskin/opac/myopac/hold_history.tt2
@@ -39,10 +39,10 @@
             <big><strong>[% l('No holds found.') %]</strong></big>
         </div>
         [% ELSE %]
-        <table style='border-collapse:collapse;width:100%;font-size:1.2em;'>
+        <table style='border-collapse:collapse;width:940px;font-size:1.2em;'>
             <thead id='acct_holds_main_header'>
                 <tr>
-                    <td><span>[% l('Title') %]</span></td>
+                    <td><span>&nbsp;[% l('Title') %]</span></td>
                     <td><span>[% l('Author') %]</span></td>
                     <td> <span>[% l('Format') %]</span></td>
                     <!-- <td width="136"><span>[% l('Pickup Location') %]</span> </td>
diff --git a/templates_tadlskin/opac/myopac/holds.tt2 b/templates_tadlskin/opac/myopac/holds.tt2
index 1a6f20954f..cb79c29707 100644
--- a/templates_tadlskin/opac/myopac/holds.tt2
+++ b/templates_tadlskin/opac/myopac/holds.tt2
@@ -89,7 +89,7 @@
                 <td width="36" align="center">
                     <input type="checkbox" onclick="var inputs=document.getElementsByTagName('input'); for (i = 0; i < inputs.length; i++) { if (inputs[i].name == 'hold_id' && !inputs[i].disabled) inputs[i].checked = this.checked;}"/>
                 </td>
-                <td width="138">
+                <td width="168">
                     <span title="Click to sort"
                         style="cursor:pointer;">
                         Title
@@ -102,7 +102,7 @@
                     <span title="Click to sort"
                         style="cursor:pointer;">Format</span>
                 </td>
-                <td width="136">
+                <td width="106">
                     <span title="Click to sort"
                         style="cursor:pointer;">Pickup Location</span>
                 </td>
@@ -128,10 +128,12 @@
                     ahr = hold.hold.hold %]
                 <tr id="acct_holds_temp" name="acct_holds_temp"
                     class="acct_holds_temp[% ahr.frozen == 't' ? ' inactive-hold' : '' %]">
-                    <td width="36" align="center" style="text-align:center;">
+                    <td width="36" align="center" style="text-align:center;vertical-align:top;">
+                        <div style="margin-top:10px;margin-bottom:10px;">
                         <input type="checkbox" name="hold_id" value="[% ahr.id %]" />
+                        </div>
                     </td>
-                    <td width="138">
+                    <td width="168" style="vertical-align:top;">
                         <div style="margin-top:10px;margin-bottom:10px;">
                             [% 
                                 title = attrs.title;
@@ -139,27 +141,30 @@
                                     title = l('[_1] ([_2])', title, hold.hold.part.label);
                                 END;
                             %]
-                            <a href="[% mkurl(ctx.opac_root _ '/record/' _ hold.hold.bre_id) %]">[% title | html %]</a>
+                            <a href="[% mkurl(ctx.opac_root _ '/record/' _ hold.hold.bre_id) %]"><img src="/opac/extras/ac/jacket/small/r/[% hold.hold.bre_id %]" style="float:left;padding-right:5px;"/>[% title | html %]</a>
                         </div>
                     </td>
-                    <td width="123">
+                    <td width="123" style="vertical-align:top;">
                         <div style="margin-top:10px;margin-bottom:10px;">
                             <a href="[% mkurl(ctx.opac_root _ '/results',
                                 {qtype => 'author', query => attrs.author.replace('[,\.:;]', '')}
                             ) %]">[% attrs.author | html %]</a>
                         </div>
                     </td>
-                    <td width="64">
+                    <td width="64" style="vertical-align:top;">
                         <div style="width:26px;height:23px;margin-top:6px;margin-bottom:6px;">
                             [% IF attrs.format_icon %]
                             <img title="[% attrs.format_label | html %]" alt="[% attrs.format_label | html %]" src="[% attrs.format_icon %]" />
                             [% END %]
                         </div>
                     </td>
-                    <td width="136">
+                    <td width="106" style="vertical-align:top;">
+                        <div style="margin-top:10px;margin-bottom:10px;">
                         [% ctx.get_aou(ahr.pickup_lib).name | html %]
+                        </div>
                     </td>
-                    <td width="104">
+                    <td width="104" style="vertical-align:top;">
+                        <div style="margin-top:10px;margin-bottom:10px;">
                         <!-- <input
                             title="Enter a date (e.g. 10/21/2010)"
                             class="hide_me" style="width:91px;"
@@ -168,30 +173,35 @@
                                 ahr.thaw_date;
                             date.format(ctx.parse_datetime(ahr.thaw_date), DATE_FORMAT);
                         END %]
+                        </div>
                     </td>
-                    <td width="106">
+                    <td width="106" style="vertical-align:top;">
+                        <div style="margin-top:10px;margin-bottom:10px;">
                         <!-- <input title="[% l('Enter a date (e.g. 10/21/2010)') %]"
                             class="hide_me" style="width:91px;"
                             name="hold_expires_box" type="text" />-->
                         [% IF ahr.expire_time;
                             date.format(ctx.parse_datetime(ahr.expire_time), DATE_FORMAT);
                         END %]
+                        </div>
                     </td>
-                    <td width="95">
+                    <td width="95" style="vertical-align:top;">
+                        <div style="margin-top:10px;margin-bottom:10px;">
                         <!-- <select name="hold_active_sel"
                             style="width:90px;" class="hide_me">
                             <option value="f">Active</option>
                             <option value="t">Suspended</option>
                         </select> -->
                         [% l(ahr.frozen == 'f' ? 'Active' : 'Suspended') %]
+                        </div>
                     </td>
-                    <td width="110">
+                    <td width="110" style="vertical-align:top;">
                         <div name="acct_holds_status"
                             style="margin-top:10px;margin-bottom:10px;">
                             [% PROCESS get_hold_status hold=hold; %]
                         </div>
                     </td>
-                    <td class="opac-auto-161">
+                    <td class="opac-auto-161" style="vertical-align:top;">
                         <a href="[% mkurl(ctx.opac_root _ '/myopac/holds/edit', {id => ahr.id}) %]">Edit</a>
                     </td>
                 </tr>
-- 
2.11.0