From: senator <lebbeous@esilibrary.com>
Date: Thu, 17 Mar 2011 16:23:13 +0000 (-0400)
Subject: make presentation better by hiding as-yet-unready features, etc
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=e46f316f3cbdd055a02b0935578a2092617a55ae;p=evergreen%2Fequinox.git

make presentation better by hiding as-yet-unready features, etc

also, made links out of the dashboard counts, but oddly the first one
("checked out") only works in IE, not Chrome or Firefox.  I suspect that
the complications of all the "position: absolute"-this and "position:
relative"-that make for fluky layout.
---

diff --git a/Open-ILS/web/css/skin/default/opac/semiauto.css b/Open-ILS/web/css/skin/default/opac/semiauto.css
index 60fd1df951..db43f5421b 100644
--- a/Open-ILS/web/css/skin/default/opac/semiauto.css
+++ b/Open-ILS/web/css/skin/default/opac/semiauto.css
@@ -24,8 +24,8 @@
 
 #lib_selector_span { display: none }
 .float-left { float: left; }
-.bookbag-name { float: left; font-weight: bold; padding-top: 5px; }
-.bookbag-share { float: left; padding: 5px 0px 0px 10px; }
+.bookbag-share { float: left; padding: 5px 0; }
+.bookbag-controls { float: left; padding: 5px 0px 0px 10px; }
 .left-corner { float: left; width: 163px; height: 30px; background: url('/images/utils-corner-mid.png') repeat-x top; }
 .float-right { float: right; }
 .opac-auto-045 { float: right; margin-right: 17px; }
diff --git a/Open-ILS/web/css/skin/default/opac/style.css b/Open-ILS/web/css/skin/default/opac/style.css
index 219a3388b2..33040f48f7 100644
--- a/Open-ILS/web/css/skin/default/opac/style.css
+++ b/Open-ILS/web/css/skin/default/opac/style.css
@@ -958,4 +958,12 @@ div.select-wrapper:hover {
 #account-update-email table { text-align: center; padding: 20px; margin-top: 30px; border-collapse: collapse; }
 #account-update-email table td { padding: 5px 15px 5px 15px; border-bottom: 1px solid #ddd; text-align: left;}
 #account-update-email-error { font-size: 1.5em; padding: 10px; border:1px solid #e9ebf3;}
-
+a.dash-link:hover { text-decoration: underline !important; }
+#list_create_table td { vertical-align: middle; padding: 0 8px; }
+#list_create_table {
+    background-color: #ccc;
+    padding-bottom: 4px;
+    margin-bottom: 10px;
+    border-bottom: 1px dotted #666;
+}
+.list-create-table-buttons input[type=image] { margin-top: 2px; }
diff --git a/Open-ILS/web/templates/default/opac/advanced.tt2 b/Open-ILS/web/templates/default/opac/advanced.tt2
index adb1807e08..8d9e125813 100644
--- a/Open-ILS/web/templates/default/opac/advanced.tt2
+++ b/Open-ILS/web/templates/default/opac/advanced.tt2
@@ -7,8 +7,8 @@
         <div id="adv_search_parent">
             <div id="adv_search_tabs">
                 <a href="#" alt="[% l('Advanced Search') %]" id="adv_search"></a>
-                <a href="#" alt="[% l('Numeric Search') %]" id="num_search"></a>
-                <a href="#" alt="[% l('Expert Search') %]" id="expert_search"></a>
+<!--                <a href="#" alt="[% l('Numeric Search') %]" id="num_search"></a>
+                    <a href="#" alt="[% l('Expert Search') %]" id="expert_search"></a> -->
             </div>
         </div>
     </div>
diff --git a/Open-ILS/web/templates/default/opac/myopac/lists.tt2 b/Open-ILS/web/templates/default/opac/myopac/lists.tt2
index 85cf1c0152..95c720c5c3 100644
--- a/Open-ILS/web/templates/default/opac/myopac/lists.tt2
+++ b/Open-ILS/web/templates/default/opac/myopac/lists.tt2
@@ -18,21 +18,42 @@
         </div>
     </div> -->
     <div id="temp_wrapper">
+<!-- new list creation -->
+<form action="[% ctx.opac_root %]/myopac/list/update" method="POST" id="create_form">
+    <h2>[% l('Create new list') %]</h2><a name="createnewlist"></a>
+<table cellpadding="0" border="0" id="list_create_table">
+    <tr>
+        <td>
+            <label for="list_create_name">[% l('Enter the name of the new list:') %]</label>
+            <input id="list_create_name" type="text" name="name" />
+            <input type="hidden" name="action" value="create" />
+        </td>
+        <td>
+            <label for="list_create_shared">[% l('Share this list?') %]</label>
+            <select name="shared" id="list_create_shared">
+                <option value="0">[% l('No') %]
+                <option value="1">[% l('Yes') %]
+            </select>
+            <a href="javascript:void(0);" onclick="alert(document.getElementById('bb_publish_text').innerHTML);"><img alt="[% l('Sharing Help') %]"
+                src="[% ctx.media_prefix %]/images/question-mark.png" /></a>
+        </td>
+        <td class="list-create-table-buttons">
+            <input type="image" alt="[% l('Submit') %]" src="[% ctx.media_prefix %]/images/btnSubmit.png"/>
+            &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
+            <a href="javascript:void(0);"
+                onclick="document.getElementById('create_form').reset(); return false"><img
+                alt="[% l('Cancel') %]"
+                src="[% ctx.media_prefix %]/images/btnCancel.png" /></a>
+        </td>
+    </tr>
+</table>
+            </form>
         [% INCLUDE "default/opac/parts/anon_list.tt2" %]
         [% IF ctx.bookbags.size %]
         <div id='acct_lists_prime'>
             [% FOR bbag IN ctx.bookbags %]
             <div id='acct_list_template'>
                 <div style="width:100%">
-                    <div class="bookbag-name">[% bbag.name %]</div>
-                    <div class="bookbag-share">
-                        [% IF bbag.pub == 't'; # XXX TODO %]
-                        <!-- <a target='_blank' name='share_list_rss'><img
-                            alt="[% l('RSS Feed') %]" border="0"
-                            src="[% ctx.media_prefix %]/images/small-rss.png"
-                            title="[% l('You are sharing this list') %]" /></a> -->
-                        [% END %]
-                    </div>
                     <form action="[% ctx.opac_root %]/myopac/list/update"
                         method="POST">
                         <div class="bookbag-share">
@@ -48,12 +69,21 @@
                     </form>
                     <form action="[% ctx.opac_root %]/myopac/list/update"
                         method="POST">
-                        <div class="bookbag-share">
+                        <div class="bookbag-controls">
                             <input type="hidden" name="list" value="[% bbag.id %]" />
                             <input type="hidden" name="action" value="delete" />
                             <input type="submit" value="[% l('Delete List') %]" />
                         </div>
                     </form>
+                    <div class="bookbag-controls"><big><strong>[% bbag.name %]</strong></big></div>
+                    <div class="bookbag-controls">
+                        [% IF bbag.pub == 't'; # XXX TODO %]
+                        <!-- <a target='_blank' name='share_list_rss'><img
+                            alt="[% l('RSS Feed') %]" border="0"
+                            src="[% ctx.media_prefix %]/images/small-rss.png"
+                            title="[% l('You are sharing this list') %]" /></a> -->
+                        [% END %]
+                    </div>
                     <div class="clear-both pad-bottom-five"></div>
                 </div>
                 <form action="[% ctx.opac_root %]/myopac/list/update"
@@ -133,34 +163,4 @@
         [% l("Bookbag successfully updated") %]
     </span>
 </div>
-<!-- new list creation -->
-<form action="[% ctx.opac_root %]/myopac/list/update" method="POST" id="create_form">
-<div style="padding-bottom: 7px;">
-    <h2>[% l('Create new list') %]</h2><a name="createnewlist"></a>
-    [% l('Enter the name of the new list:') %]<br/>
-    <input type="hidden" name="action" value="create" />
-    <input type="text" name="name" />
-</div>
-
-<table cellpadding="0" cellspacing="10" border="0">
-    <tr>
-        <td>
-            [% l('Share this list?') %]
-            <a href="#"><img alt="[% l('Sharing Help') %]"
-                src="[% ctx.media_prefix %]/images/question-mark.png" /></a>
-        </td>
-        <td>
-            <input type="radio" value="0" name="shared" id="shared_0" checked="checked" /><label for="shared_0">[% l('No') %]</label>
-            <br/>
-            <input type="radio" value="1" name="shared" id="shared_1"/><label for="shared_1">[% l('Yes') %]</label>
-        </td>
-    </tr>
-</table>
-<input type="image" alt="[% l('Submit') %]" src="[% ctx.media_prefix %]/images/btnSubmit.png"/>
-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
-<a href="javascript:void(0);"
-    onclick="document.getElementById('create_form').reset(); return false"><img
-    alt="[% l('Cancel') %]"
-    src="[% ctx.media_prefix %]/images/btnCancel.png" /></a>
-</form>
 [% END %]
diff --git a/Open-ILS/web/templates/default/opac/parts/anon_list.tt2 b/Open-ILS/web/templates/default/opac/parts/anon_list.tt2
index e1e72fe552..cc06a0d351 100644
--- a/Open-ILS/web/templates/default/opac/parts/anon_list.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/anon_list.tt2
@@ -8,9 +8,9 @@
                             [% l('Temporary List') %]
                         </td>
                         <td>
-                            <a href="#"><img
+                            <!-- <a href="#"><img
                                 alt="[% l('Anonymous List Help') %]"
-                                src="[% ctx.media_prefix %]/images/question-mark.png" /></a>
+                                src="[% ctx.media_prefix %]/images/question-mark.png" /></a>-->
                         </td>
                     </tr>
                 </table>
diff --git a/Open-ILS/web/templates/default/opac/parts/result/table.tt2 b/Open-ILS/web/templates/default/opac/parts/result/table.tt2
index 4b14e04c75..90b25481e0 100644
--- a/Open-ILS/web/templates/default/opac/parts/result/table.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/result/table.tt2
@@ -199,13 +199,13 @@
                                                                 src="[% ctx.media_prefix %]/images/clipboard.png" />
                                                         </form>
                                                         </div>
-                                                        <div style="padding-top:7px;" class="results_aux_utils">
+                                                        <!-- <div style="padding-top:7px;" class="results_aux_utils">
                                                             <a title="Reviews and More" target="_blank" class="no-dec"
                                                                 name="reviews_and_more" href="javascript:;"><img
                                                                 alt="reviews &amp; more"
                                                                 src="[% ctx.media_prefix %]/images/starz.png" /> <span
                                                                     style="position:relative;top:-5px;">Reviews &amp; More</span></a>
-                                                        </div>
+                                                        </div> -->
                                                     </div>
                                                     <div style="float:right;margin-right:17px;">
                                                     [%  key = attrs.mattype;
diff --git a/Open-ILS/web/templates/default/opac/parts/topnav.tt2 b/Open-ILS/web/templates/default/opac/parts/topnav.tt2
index f2401d8ba5..15a98af292 100644
--- a/Open-ILS/web/templates/default/opac/parts/topnav.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/topnav.tt2
@@ -62,28 +62,28 @@
                                         <div class="pos-abs">
                                             <div class="dash-pos-out">
                                                 <div class="dash-align-out">
-                                                    <span id="dash_checked">[% ctx.user_stats.checkouts.total_out %]</span> [% l("Checked Out") %]
+                                                    <a class="dash-link" href="[% ctx.opac_root %]/myopac/circs"><span id="dash_checked">[% ctx.user_stats.checkouts.total_out %]</span> [% l("Checked Out") %]</a>
                                                 </div>
                                             </div>
                                         </div>
                                         <div class="pos-abs">
                                             <div class="dash-pos-holds">
                                                 <div class="dash-align-holds">
-                                                    <span id="dash_holds">[% ctx.user_stats.holds.total %]</span> [% l("On Hold") %]
+                                                    <a class="dash-link" href="[% ctx.opac_root %]/myopac/holds"><span id="dash_holds">[% ctx.user_stats.holds.total %]</span> [% l("On Hold") %]</a>
                                                 </div>
                                             </div>
                                         </div>
                                         <div class="pos-abs">
                                             <div class="dash-pos-pickup">
                                                 <div class="dash-align-pickup">
-                                                    <span id="dash_pickup">[% ctx.user_stats.holds.ready %]</span> [% l("Ready for Pickup") %]
+                                                    <a class="dash-link" href="[% ctx.opac_root %]/myopac/holds?available=1"><span id="dash_pickup">[% ctx.user_stats.holds.ready %]</span> [% l("Ready for Pickup") %]</a>
                                                 </div>
                                             </div>
                                         </div>
                                         <div class="pos-abs">
                                             <div class="dash-pos-fines">
                                                 <div class="dash-align-fines">
-                                                    <span id="dash_fines">[% money(ctx.user_stats.fines.balance_owed) %]</span> [% l("Fines") %]
+                                                    <a class="dash-link" href="[% ctx.opac_root %]/myopac/main"><span id="dash_fines">[% money(ctx.user_stats.fines.balance_owed) %]</span> [% l("Fines") %]</a>
                                                 </div>
                                             </div>
                                         </div>