From a0a0d9a0c4eec6d608c041bfb5797f1eff6d6146 Mon Sep 17 00:00:00 2001
From: Dan Scott <dscott@laurentian.ca>
Date: Wed, 18 Sep 2013 10:57:52 -0400
Subject: [PATCH] Make record actions available in mobile record details

Use the format label approach and push the actions beside the cover art
(if any) so that users can do actions like place hold in a mobile UI.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
---
 Open-ILS/src/templates/opac/css/style.css.tt2      | 31 ++++++++--------
 .../src/templates/opac/parts/record/summary.tt2    | 41 +++++++++++-----------
 2 files changed, 35 insertions(+), 37 deletions(-)

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 00d7cb41bd..722e7ed885 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -1643,6 +1643,10 @@ a.preflib_change {
     display: none;
 }
 
+.rdetail_authors_div {
+    margin-bottom: 1em;
+}
+
 @media only screen and (max-width: 600px) {
     body {
         font-size: 12pt;
@@ -1839,21 +1843,6 @@ a.preflib_change {
     #main-content {
         margin-left: 1px;
     }
-    #rdetail_actions_div {
-        float: none !important;
-    }
-    #rdetail_actions_div div:nth-child(2), #rdetail_actions_div div:nth-child(3) {
-        display: none;
-    }
-    div#rdetail_title_div .format_icon {
-        float: none;
-        display: inline;
-        margin-right: 5px;
-    }
-    #rdetail_title {
-        float: none;
-        display: inline;
-    }
     #rdetails_status thead {
         display: none;
     }
@@ -1897,5 +1886,15 @@ a.preflib_change {
         position: relative;                                                                                                                           
         width: 300px;
     }
-
+    #rdetail_format_label {
+        float: left;
+    }
+    .rdetail_aux_utils {
+        padding: 0px;
+        border: none;
+    }
+    div#rdetail_actions_div {
+        float: left;
+        background: [% css_colors.background %];
+    }
 }
diff --git a/Open-ILS/src/templates/opac/parts/record/summary.tt2 b/Open-ILS/src/templates/opac/parts/record/summary.tt2
index b9a91a89ad..2e0e2222a0 100644
--- a/Open-ILS/src/templates/opac/parts/record/summary.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/summary.tt2
@@ -9,6 +9,21 @@
 
 [%-# This holds the record summary information %]
 <div id="rdetail_summary_header">
+    <div id='rdetail_title_div'>
+        <h1 id='rdetail_title' property="name">[% attrs.title_extended | html %]</h1>
+        [%-
+            FOR link880 IN attrs.graphic_titles;
+                FOR alt IN link880.graphic;
+                    '<h2 class="graphic880"';
+                    IF alt.dir;
+                        ' dir="' _ alt.dir _ '"';
+                    END;
+                    '>'; alt.value | html; '</h2>';
+                END;
+            END;
+        -%]
+        [%- INCLUDE "opac/parts/record/authors.tt2" %]
+    </div>
     <div id="rdetail_image_div">
         [% ident = attrs.isbn_clean || attrs.upc; IF ident; %]
         <a href='[% ctx.media_prefix %]/opac/extras/ac/jacket/large/[% ident | uri %]'><img
@@ -17,7 +32,11 @@
         [% END %]
         <br />
     </div>
-
+    [%- IF attrs.format_label %]
+    <div id="rdetail_format_label">
+        [%- attrs.format_label -%]
+    </div>
+    [%- END %]
     <div id="rdetail_actions_div">
         [%- search_ou = ctx.search_ou;
             IF ctx.place_unfillable ||
@@ -55,26 +74,6 @@
             [%- INCLUDE 'opac/parts/record/refworks.tt2' %]
         [%- END %]
     </div>
-    <div id='rdetail_title_div'>
-        [%- IF attrs.format_icon %]
-        <div class="format_icon">
-            <img alt="[% attrs.format_label %]" title="[% attrs.format_label | html %]" src="[% attrs.format_icon %]" />
-        </div>
-        [%- END %]
-        <h1 id='rdetail_title' property="name">[% attrs.title_extended | html %]</h1>
-        [%-
-            FOR link880 IN attrs.graphic_titles;
-                FOR alt IN link880.graphic;
-                    '<h2 class="graphic880"';
-                    IF alt.dir;
-                        ' dir="' _ alt.dir _ '"';
-                    END;
-                    '>'; alt.value | html; '</h2>';
-                END;
-            END;
-        -%]
-        [%- INCLUDE "opac/parts/record/authors.tt2" %]
-    </div>
 </div>
 
 [%- IF openurl.enabled == 'true';
-- 
2.11.0