[open-ils-commits] [GIT] Evergreen ILS branch rel_2_12 updated. a724acf4083e070118cacf9562435f943321fe48

Evergreen Git git at git.evergreen-ils.org
Fri Jun 9 12:06:57 EDT 2017


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_12 has been updated
       via  a724acf4083e070118cacf9562435f943321fe48 (commit)
       via  827790e8c879e5f8fc4e413b88ef1d684e97712f (commit)
       via  372912d54af658b0e349653163a8100f72c180ef (commit)
       via  2702937a62b3cc1d992eba9c29d2b8d28e2a7249 (commit)
       via  a843c31b0e430f99d6fa14ef5f688409f2a45122 (commit)
      from  07e2652fd5ea4c07baf4ce690aebb4445c0aa3df (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 a724acf4083e070118cacf9562435f943321fe48
Author: Galen Charlton <gmc at equinoxinitiative.org>
Date:   Wed May 24 21:22:11 2017 -0400

    LP#1681943: show all list fields in mobile view
    
    In mobile view, "rotate" the temporary and permanent list
    tables 90 degrees and put the column headings on the left/right
    side.  data-label attributes are used to stash copies of the
    column labels; the end result is similar to how the current
    circulations table is displayed on narrow screens.
    
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    Signed-off-by: Terran McCanna <tmccanna at georgialibraries.org>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 173d2b2..f520d37 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -2061,6 +2061,7 @@ table.bookbag-specific {
     border-right: 1px solid [% css_colors.accent_dark %];
     border-bottom: 1px solid [% css_colors.accent_medium %];
     margin-bottom: 2ex;
+    width: 100%;
 }
 .bookbag-share {
     [% IF rtl == 't' -%]
@@ -2700,6 +2701,10 @@ a.preflib_change {
     .bookbag-specific div.sort select {
         width:180px;
     }
+    .bookbag-specific tr {
+        display: block;
+        border-bottom: 1px solid [% css_colors.border_standard %];
+    }
     .bookbag-specific td.list_checkbox {
         [% IF rtl == 't' -%]
         padding-right: 0px !important;
@@ -2708,11 +2713,21 @@ a.preflib_change {
         [% END -%]
     }
     .bookbag-specific td.list_entry {
+        display: inline-block;
         min-width: 5em;
         [% IF rtl == 't' -%]
-        padding-right: 5px !important;
+        padding-right: 40% !important;
+        [% ELSE -%]
+        padding-left: 40% !important;
+        [% END -%]
+    }
+    .bookbag-specific td.list_entry:before {
+        content: attr(data-label);
+        position: absolute;
+        [% IF rtl == 't' -%]
+        right: 10px;
         [% ELSE -%]
-        padding-left: 5px !important;
+        left: 10px;
         [% END -%]
     }
     .bbag-navigate-list {
diff --git a/Open-ILS/src/templates/opac/myopac/lists.tt2 b/Open-ILS/src/templates/opac/myopac/lists.tt2
index b12b9dc..510fea2 100644
--- a/Open-ILS/src/templates/opac/myopac/lists.tt2
+++ b/Open-ILS/src/templates/opac/myopac/lists.tt2
@@ -347,16 +347,16 @@
                     <td class="list_checkbox">
                         <input type="checkbox" name="selected_item" value="[% item.id %]" bbag='[% bbag.id %]'/>
                     </td>
-                    <td class="list_entry">
+                    <td class="list_entry" data-label="[% l('Title') %]">
                         <a href="[% mkurl(ctx.opac_root _ '/record/' _ rec_id, {}, ['edit_notes', 'bbid']) %]">[% attrs.title | html %]</a>
                     </td>
-                    <td class="list_entry">
-                        <a href="[%- 
+                    <td class="list_entry" data-label="[% l('Author(s)') %]">
+                        <a href="[%-
                             authorquery = attrs.author | replace('[,\.:;]', '');
                             mkurl(ctx.opac_root _ '/results', {qtype => 'author', query => authorquery}, ['page', 'bbid', 'edit_notes'])
                             -%]">[% attrs.author | html %]</a>
                     </td>
-                    <td class="list_entry mobile_hide">
+                    <td class="list_entry" data-label="[% l('Local Call Number') %]">
                         [% 
                             copy = attrs.holdings.0;
                             IF copy;
@@ -373,21 +373,21 @@
                             END;
                         %]
                     </td>
-                    <td class="list_entry mobile_hide">
+                    <td class="list_entry" data-label="[% l('Publication Date') %]">
                           [% attrs.pubdate | html %]
                     </td>
-                    <td class="list_entry mobile_hide">
+                    <td class="list_entry" data-label="[% l('Format') %]">
                           [% attrs.format_label | html %]
                     </td>
                     [% IF CGI.param("edit_notes") == bbag.id %]
-                    <td class="list_entry mobile_hide">
+                    <td class="list_entry" data-label="[% l('Notes') %]">
                         [% FOR note IN item.notes %]
                         <input type="text" name="note-[% note.id %]" value="[% note.note | html %]" />
                         [% END %]
                         <input type="text" name="item-[% item.id %]" />
                     </td>
                     [% ELSE %]
-                    <td class="list_entry mobile_hide">
+                    <td class="list_entry" data-label="[% l('Notes') %]">
                         [% FOR note IN item.notes %]
                         <div>[% note.note | html %]</div>
                         [% END %]
diff --git a/Open-ILS/src/templates/opac/parts/anon_list.tt2 b/Open-ILS/src/templates/opac/parts/anon_list.tt2
index dab23bf..9ec6d58 100644
--- a/Open-ILS/src/templates/opac/parts/anon_list.tt2
+++ b/Open-ILS/src/templates/opac/parts/anon_list.tt2
@@ -30,7 +30,7 @@
             <input class="opac-button" type="submit" value="[% l('Go') %]" />
         </div>
         <div class="bbag-content">
-            <table class="bookbag-specific table_no_cell_pad table_no_border_space table_no_border" width="100%">
+            <table class="bookbag-specific table_no_cell_pad table_no_border_space table_no_border">
                 <thead id="acct_list_header_anon">
                     <tr>
                         <td class='list_checkbox'>
@@ -52,8 +52,8 @@
                         <td class="list_checkbox">
                             <input type="checkbox" name="record" value="[% item %]" />
                         </td>
-                        <td class="list_entry"><a href="[% mkurl(ctx.opac_root _ '/record/' _ item, {}, ['edit_notes', 'id']) %]">[% attrs.title | html %]</a></td>
-                        <td class="list_entry"><a href="[%- 
+                        <td class="list_entry" data-label="[% l('Title') %]"><a href="[% mkurl(ctx.opac_root _ '/record/' _ item, {}, ['edit_notes', 'id']) %]">[% attrs.title | html %]</a></td>
+                        <td class="list_entry" data-label="[% l('Author(s)') %]"><a href="[%-
                             authorquery = attrs.author | replace('[,\.:;]', '');
                             mkurl(
                                 ctx.opac_root _ '/results',
@@ -61,7 +61,7 @@
                                 ['page', 'id', 'edit_notes']
                             )
                         -%]">[% attrs.author | html %]</a></td>
-                        <td class="list_entry mobile_hide">
+                        <td class="list_entry" data-label="[% l('Local Call Number') %]">
                         [% 
                             copy = attrs.holdings.0;
                             IF copy;

commit 827790e8c879e5f8fc4e413b88ef1d684e97712f
Author: Ben Shum <ben at evergreener.net>
Date:   Mon May 1 22:42:11 2017 -0400

    LP#1681943: Tweak for RTL handling for responsive my lists
    
    Needs some additional changes to support RTL for responsive my lists.
    
    Signed-off-by: Ben Shum <ben at evergreener.net>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    Signed-off-by: Terran McCanna <tmccanna at georgialibraries.org>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index b247cc1..173d2b2 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -1946,11 +1946,19 @@ a.opac-button-header:hover, #dash_wrapper a.opac-button:hover {
     [% END -%]
 }
 .bookbag-controls-title-block {
+    [% IF rtl == 't' -%]
+    float:right;
+    [% ELSE -%]
     float:left;
+    [% END -%]
     width:40%;
 }
 .bookbag-controls-button-block {
+    [% IF rtl == 't' -%]
+    float:right;
+    [% ELSE -%]
     float:left;
+    [% END -%]
     width:50%;
 }
 h2.bookbag-name { margin-bottom: 0em; padding-bottom: 0em; }
@@ -1996,25 +2004,57 @@ h2.bookbag-name { margin-bottom: 0em; padding-bottom: 0em; }
     max-width: 40em;
 }
 .bbag-edit-desc-label {
+    [% IF rtl == 't' -%]
+    float:right;
+    [% ELSE -%]
     float:left;
+    [% END -%]
     width:8em;
 }
 .bbag-edit-desc-save {
     clear:both;
     margin-bottom:10px;
 }
-#bbag-edit-description { width: 20em; float:left;}
-#bbag-edit-name { width: 20em; float: left; }
+#bbag-edit-description {
+    width: 20em;
+    [% IF rtl == 't' -%]
+    float:right;
+    [% ELSE -%]
+    float:left;
+    [% END -%]
+}
+#bbag-edit-name {
+    width: 20em;
+    [% IF rtl == 't' -%]
+    float: right;
+    [% ELSE -%]
+    float: left;
+    [% END -%]
+}
 .bbag-action {
+    [% IF rtl == 't' -%]
+    margin-right: 15px;
+    [% ELSE -%]
     margin-left: 15px;
+    [% END -%]
 }
 .bbag-navigate-list {
+    [% IF rtl == 't' -%]
+    padding-right: 32px;
+    float:right;
+    [% ELSE -%]
     padding-left: 32px;
     float:left;
+    [% END -%]
 }
 .bbag-navigate-list-pages {
+    [% IF rtl == 't' -%]
+    padding-right: 10px;
+    float:right;
+    [% ELSE -%]
     padding-left: 10px;
     float:left;
+    [% END -%]
 }
 textarea { font-family: sans-serif; }
 table.bookbag-specific {
@@ -2581,34 +2621,61 @@ a.preflib_change {
        font-size: [% css_fonts.size_base %];
     }
     .bookbag-controls-title-block {
+        [% IF rtl == 't' -%]
+        clear:right;
+        [% ELSE -%]
         clear:left;
+        [% END -%]
         width:90%;
     }
     .bookbag-controls-button-block {
+        [% IF rtl == 't' -%]
+        clear:right;
+        [% ELSE -%]
         clear:left;
+        [% END -%]
         width;90%;
     }
     .bookbag-specific {
+        [% IF rtl == 't' -%]
+        margin-right: 0px;
+        [% ELSE -%]
         margin-left: 0px;
+        [% END -%]
     }
     .bookbag-specific div.sort {
+        [% IF rtl == 't' -%]
+        float: right;
+        text-align: right;
+        [% ELSE -%]
         float: left;
+        text-align: left;
+        [% END -%]
         width: 95%;
         margin: 5px 0px 5px 0px;
-        text-align: left;
         border: 1px solid [% css_colors.accent_light %];
         padding:5px;
     }
     .bookbag-specific div.meta {
+        [% IF rtl == 't' -%]
+        float: right;
+        margin-right:0px;
+        text-align: right;
+        [% ELSE -%]
         float: left;
-        width: 95%;
         margin-left:0px;
+        text-align: left;
+        [% END -%]
+        width: 95%;
         margin-bottom:5px;
         padding:5px;
-        text-align: left;
     }
     #bbag-edit-name {
+        [% IF rtl == 't' -%]
+        float: right;
+        [% ELSE -%]
         float: left;
+        [% END -%]
         width: 220px;
     }
     #bbag-edit-description {
@@ -2620,7 +2687,11 @@ a.preflib_change {
         border:1px solid [% css_colors.accent_light %];
     }
     .bbag-action {
+        [% IF rtl == 't' -%]
+        margin-right:0px;
+        [% ELSE -%]
         margin-left:0px;
+        [% END -%]
         margin-bottom:5px;
     }
     .bbag-action-field {
@@ -2630,18 +2701,31 @@ a.preflib_change {
         width:180px;
     }
     .bookbag-specific td.list_checkbox {
+        [% IF rtl == 't' -%]
+        padding-right: 0px !important;
+        [% ELSE -%]
         padding-left: 0px !important;
+        [% END -%]
     }
     .bookbag-specific td.list_entry {
         min-width: 5em;
+        [% IF rtl == 't' -%]
+        padding-right: 5px !important;
+        [% ELSE -%]
         padding-left: 5px !important;
+        [% END -%]
     }
     .bbag-navigate-list {
         display: none;
     }
     .bbag-navigate-list-pages {
+        [% IF rtl == 't' -%]
+        text-align:left;
+        float:left;
+        [% ELSE -%]
         text-align:right;
         float:right;
+        [% END -%]
     }
     #dash_wrapper div {
         background: transparent;

commit 372912d54af658b0e349653163a8100f72c180ef
Author: Ben Shum <ben at evergreener.net>
Date:   Mon May 1 22:27:10 2017 -0400

    LP#1681943: Use variables for CSS colors
    
    There were some hardcoded references to #ccc in the stylesheet, and
    those should be variables like [% css_colors.accent_light %] instead.
    
    Signed-off-by: Ben Shum <ben at evergreener.net>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    Signed-off-by: Terran McCanna <tmccanna at georgialibraries.org>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 03240f3..b247cc1 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -594,7 +594,7 @@ span#rdetail_hold_counts p {
 .rdetail_aux_utils {
     padding-bottom: 6px;
     [% IF rtl == 't' -%]
-    border-right: 1px dotted #ccc;
+    border-right: 1px dotted [% css_colors.accent_light %];
     padding-right:17px;
     padding-left:70px;
     [%- ELSE %]
@@ -2596,7 +2596,7 @@ a.preflib_change {
         width: 95%;
         margin: 5px 0px 5px 0px;
         text-align: left;
-        border: 1px #ccc solid;
+        border: 1px solid [% css_colors.accent_light %];
         padding:5px;
     }
     .bookbag-specific div.meta {
@@ -2617,7 +2617,7 @@ a.preflib_change {
     }
     .bbag-content {
         padding:5px;
-        border:1px #ccc solid;
+        border:1px solid [% css_colors.accent_light %];
     }
     .bbag-action {
         margin-left:0px;

commit 2702937a62b3cc1d992eba9c29d2b8d28e2a7249
Author: Galen Charlton <gmc at equinoxinitiative.org>
Date:   Wed Apr 26 09:21:06 2017 -0400

    LP#1681943: tweak background color of edit list description box
    
    This makes it consistent with the 'create new list' box
    and improves the contrast of the inputs.
    
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    Signed-off-by: Ben Shum <ben at evergreener.net>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    Signed-off-by: Terran McCanna <tmccanna at georgialibraries.org>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 0375287..03240f3 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -1975,6 +1975,7 @@ h2.bookbag-name { margin-bottom: 0em; padding-bottom: 0em; }
     width: 40%;
 }
 .bookbag-specific div.meta {
+    background-color: [% css_colors.accent_light %];
     [% IF rtl == 't' -%]
     float: left;
     text-align: left;

commit a843c31b0e430f99d6fa14ef5f688409f2a45122
Author: Terran McCanna <tmccanna at georgialibraries.org>
Date:   Tue Apr 11 17:42:54 2017 -0400

    LP#1681943 Improve Responsive Design in My Lists
    
    - Moved action dropdown out of header row so that it is no longer
    hidden on mobile view.
    - Hid less essential columns on mobile view, leaving only title / author.
    - Sized and lined up buttons more neatly on both mobile and normal view.
    - Added a title to the section for editing a list title and description.
    
    Signed-off-by: Terran McCanna <tmccanna at georgialibraries.org>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    
    Conflicts:
    	Open-ILS/src/templates/opac/css/style.css.tt2
    	Open-ILS/src/templates/opac/parts/anon_list.tt2
    
    Signed-off-by: Ben Shum <ben at evergreener.net>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index aba12de..0375287 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -1676,6 +1676,13 @@ a.dash-link:hover { text-decoration: underline !important; }
     padding-bottom: 4px;
     margin-bottom: 10px;
     border-bottom: 1px dotted [% css_colors.accent_medium %];
+    width: 100%;
+}
+.list_create_table_label {
+    width: 30%;
+}
+#list_description, #list_create_name {
+        width: 500px;
 }
 .list-create-table-buttons input[type=image] { margin-top: 2px; }
 .result_table_format_cell { padding: 0px 10px; text-align: center; }
@@ -1938,6 +1945,14 @@ a.opac-button-header:hover, #dash_wrapper a.opac-button:hover {
     margin-right: 5em;
     [% END -%]
 }
+.bookbag-controls-title-block {
+    float:left;
+    width:40%;
+}
+.bookbag-controls-button-block {
+    float:left;
+    width:50%;
+}
 h2.bookbag-name { margin-bottom: 0em; padding-bottom: 0em; }
 .bookbag-share .fixed { min-width: 4em; }
 .bookbag-specific {
@@ -1951,11 +1966,13 @@ h2.bookbag-name { margin-bottom: 0em; padding-bottom: 0em; }
     [% IF rtl == 't' -%]
     float: right;
     text-align: right;
+    margin-right: 15px;
     [% ELSE -%]
     float: left;
     text-align: left;
+    margin-left: 15px;
     [% END -%]
-    width: 45%;
+    width: 40%;
 }
 .bookbag-specific div.meta {
     [% IF rtl == 't' -%]
@@ -1966,6 +1983,9 @@ h2.bookbag-name { margin-bottom: 0em; padding-bottom: 0em; }
     text-align: right;
     [% END -%]
     width: 54%;
+    border: 1px solid [% css_colors.accent_light %];
+    padding: 5px;
+    margin-top: 5px;
 }
 #bbag-name-desc-form tr th { vertical-align: middle; }
 #bbag-name-desc-form .saver { vertical-align: middle; text-align: center; }
@@ -1974,8 +1994,27 @@ h2.bookbag-name { margin-bottom: 0em; padding-bottom: 0em; }
     font-style: italic;
     max-width: 40em;
 }
-#bbag-edit-description { width: 20em; }
-#bbag-edit-name { width: 20em; }
+.bbag-edit-desc-label {
+    float:left;
+    width:8em;
+}
+.bbag-edit-desc-save {
+    clear:both;
+    margin-bottom:10px;
+}
+#bbag-edit-description { width: 20em; float:left;}
+#bbag-edit-name { width: 20em; float: left; }
+.bbag-action {
+    margin-left: 15px;
+}
+.bbag-navigate-list {
+    padding-left: 32px;
+    float:left;
+}
+.bbag-navigate-list-pages {
+    padding-left: 10px;
+    float:left;
+}
 textarea { font-family: sans-serif; }
 table.bookbag-specific {
     border-right: 1px solid [% css_colors.accent_dark %];
@@ -1985,21 +2024,22 @@ table.bookbag-specific {
 .bookbag-share {
     [% IF rtl == 't' -%]
     float: right;
+    padding: 0px 10px 0px 0px;
     [% ELSE -%]
     float: left;
+    padding: 0px 0px 0px 10px;
     [% END -%]
-    padding: 5px 0;
 }
+.bookbag-share .fixed { min-width: 6em; }
 .bookbag-controls {
     [% IF rtl == 't' -%]
     float: right;
-    padding: 5px 10px 0px 0px;
+    padding: 0px 10px 0px 0px;
     [% ELSE -%]
     float: left;
-    padding: 5px 0px 0px 10px;
+    padding: 0px 0px 0px 10px;
     [% END -%]
 }
-
 .bookbag-specific td.list_checkbox {
     [% IF rtl == 't' -%]
     padding-right: 10px !important;
@@ -2481,6 +2521,9 @@ a.preflib_change {
          display:none;
      }
     .radio-parts-selection { width: 90%; }
+    #list_description, #list_create_name {
+        width: 300px;
+    }
 }
 
 @media only screen and (max-width: 600px) {
@@ -2536,7 +2579,69 @@ a.preflib_change {
        padding: 2px 4px 3px 4px;
        font-size: [% css_fonts.size_base %];
     }
-
+    .bookbag-controls-title-block {
+        clear:left;
+        width:90%;
+    }
+    .bookbag-controls-button-block {
+        clear:left;
+        width;90%;
+    }
+    .bookbag-specific {
+        margin-left: 0px;
+    }
+    .bookbag-specific div.sort {
+        float: left;
+        width: 95%;
+        margin: 5px 0px 5px 0px;
+        text-align: left;
+        border: 1px #ccc solid;
+        padding:5px;
+    }
+    .bookbag-specific div.meta {
+        float: left;
+        width: 95%;
+        margin-left:0px;
+        margin-bottom:5px;
+        padding:5px;
+        text-align: left;
+    }
+    #bbag-edit-name {
+        float: left;
+        width: 220px;
+    }
+    #bbag-edit-description {
+        width: 220px;
+        margin-top:5px;
+    }
+    .bbag-content {
+        padding:5px;
+        border:1px #ccc solid;
+    }
+    .bbag-action {
+        margin-left:0px;
+        margin-bottom:5px;
+    }
+    .bbag-action-field {
+        width:230px;
+    }
+    .bookbag-specific div.sort select {
+        width:180px;
+    }
+    .bookbag-specific td.list_checkbox {
+        padding-left: 0px !important;
+    }
+    .bookbag-specific td.list_entry {
+        min-width: 5em;
+        padding-left: 5px !important;
+    }
+    .bbag-navigate-list {
+        display: none;
+    }
+    .bbag-navigate-list-pages {
+        text-align:right;
+        float:right;
+    }
     #dash_wrapper div {
         background: transparent;
         padding: 0px;
@@ -2579,6 +2684,9 @@ a.preflib_change {
     #myopac_sum_fines {
         display: none;
     }
+    #list_description, #list_create_name {
+        width: 170px;
+    }
     .results_header_lbl {
         display: none;
     }
diff --git a/Open-ILS/src/templates/opac/myopac/lists.tt2 b/Open-ILS/src/templates/opac/myopac/lists.tt2
index b2c26d2..b12b9dc 100644
--- a/Open-ILS/src/templates/opac/myopac/lists.tt2
+++ b/Open-ILS/src/templates/opac/myopac/lists.tt2
@@ -19,7 +19,7 @@
         <h1>[% l('Create New List') %]</h1><a name="createnewlist"></a>
         <table class="table_no_cell_pad" id="list_create_table">
             <tr>
-                <td>
+                <td class="list_create_table_label">
                     <label for="list_create_name">[% l('Enter the name of the new list:') %]</label>
                 </td>
                 <td>
@@ -33,8 +33,20 @@
                     <input type="hidden" name="where_from" value="[% ctx.where_from %]" />
                     [% END %]
                 </td>
+            </tr>
+            <tr>
+                <td class="list_create_table_label">
+                    <label for="list_description">[% l("List description (optional):") %]</label>
+                </td>
                 <td>
+                    <textarea name="description" id="list_description"></textarea>
+                </td>
+            </tr>
+            <tr>
+                <td class="list_create_table_label">
                     <label for="list_create_shared">[% l('Share this list?') %]</label>
+                </td>
+                <td>
                     <select name="shared" id="list_create_shared">
                         <option value="0">[% l('No') %]
                         <option value="1">[% l('Yes') %]
@@ -46,26 +58,21 @@
                             src="[% ctx.media_prefix %]/images/question-mark.png" />
                     </a>
                 </td>
+            </tr>
+            <tr>
+                <td> </td>
                 <td class="list-create-table-buttons">
-                    <input type="submit"
-                        value="[% l('Submit') %]"
-                        alt="[% l('Submit') %]"
-                        class="opac-button"/>
-                         
                     <input type="reset"
                         value="[% l('Cancel') %]"
                         alt="[% l('Cancel') %]"
                         class="opac-button" />
+                       
+                    <input type="submit"
+                        value="[% l('Submit') %]"
+                        alt="[% l('Submit') %]"
+                        class="opac-button"/>
                 </td>
             </tr>
-            <tr>
-                <td class="text-right-top">
-                    <label for="list_description">[% l("List description (optional):") %]</label>
-                </td>
-                <td colspan="3">
-                    <textarea cols="40" rows="3" name="description"
-                        id="list_description"></textarea>
-                </td>
         </table>
     </form>
 
@@ -125,7 +132,7 @@
     <div id='acct_lists_prime'>
         [% FOR bbag IN ctx.bookbags %]
         <div class="bookbag-controls-holder">
-            <div class="bookbag-controls most">
+            <div class="bookbag-controls-title-block">
                 [% baseurl = ctx.opac_root _ '/myopac/lists';
                 IF bbag.id != CGI.param("bbid");
                     url = mkurl(baseurl,{bbid => bbag.id, item_page => 1},['edit_notes','sort']);
@@ -137,6 +144,7 @@
                 <h2 class="bookbag-name"><a title="[% ltitle %]" href="[% url %]">[% bbag.name | html %]</a></h2>
                 [% IF bbag.description %]<div class="bookbag-description">[% bbag.description | html %]</div>[% END %]
             </div>
+            <div class="bookbag-controls-button-block">
             [% IF ctx.add_rec %]
             <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update', {}, 1) %]" method="post">
                 <div class="bookbag-controls">
@@ -220,8 +228,6 @@
                     alt="[% l('RSS Feed') %]" border="0"
                     src="[% ctx.media_prefix %]/images/small-rss.png"/></a>
                 [% END %]
-            </div>
-            <div class="bookbag-controls">
                 [% IF bbag.pub == 't'; %]
                 <a href='[%-
                     mkurl(
@@ -232,21 +238,24 @@
                 -%]'>[% l('HTML View') %]</a>
                 [% END %]
             </div>
+            </div>
             <div class="clear-both pad-bottom-five"></div>
         </div>
         [% IF CGI.param("bbid") == bbag.id %]
         <div class="bookbag-specific">
-            <div class="sort">
-                <form method="get">
-                    <label for="opac.result.sort">[% l("Sort list items by: ") %]</label>
-                    [%- INCLUDE "opac/parts/preserve_params.tt2" params=['loc', 'query', 'qtype']; %]
-                    [% INCLUDE "opac/parts/filtersort.tt2"
-                        value=CGI.param('sort') mode='bookbag' %]
-                    <input type="hidden" name="bbid"
-                        value="[% CGI.param('bbid') | html %]" />
-                    <input type="submit" value="[% l('Sort') %]" />
-                </form>
-            </div>
+            [% IF bbag.items.size %]
+                <div class="sort">
+                    <form method="get">
+                        <label for="opac.result.sort">[% l("Sort list items by: ") %]</label>
+                        [%- INCLUDE "opac/parts/preserve_params.tt2" params=['loc', 'query', 'qtype']; %]
+                        [% INCLUDE "opac/parts/filtersort.tt2"
+                            value=CGI.param('sort') mode='bookbag' %]
+                        <input type="hidden" name="bbid"
+                            value="[% CGI.param('bbid') | html %]"/>
+                        <input type="submit" class="opac-button" value="[% l('Sort') %]" />
+                    </form>
+                </div>
+            [% END %]
             <div class="meta">
                 <form method="post">
                     <input type="hidden" name="bbid" value="[% bbag.id %]" />
@@ -254,29 +263,23 @@
                     <input type="hidden" name="limit" value="[% limit %]" />
                     <input type="hidden" name="offset" value="[% offset %]" />
                     [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
-                    <table id="bbag-name-desc-form">
-                        <tr>
-                            <th>
+                            <p style="font-weight:bold;">[% l('Edit List Description') %]</p>
+                            <div class="bbag-edit-desc-label">
                                 <label for="bbag-edit-name">[% l('Name:') %]</label>
-                            </th>
-                            <td>
-                                <input name="name" type="text"
+                            </div>
+                            <input name="name" type="text"
                                     value="[% bbag.name | html %]"
                                     id="bbag-edit-name" />
-                            </td>
-                            <td rowspan="2" class="saver">
-                                [% l("Save changes to name or description?") %]<br />
-                                <input type="submit" value="[% l('Save') %]" />
-                            </td>
-                        </tr>
-                        <tr>
-                            <th><label for="bbag-edit-description">[% l('Description:') %]</label></th>
-                            <td>
-                                <textarea name="description"
+                            <div style="clear:both;" />
+                            <div class="bbag-edit-desc-label">
+                                <label for="bbag-edit-description">[% l('Description:') %]</label>
+                            </div>
+                            <textarea name="description"
                                     id="bbag-edit-description">[% bbag.description | html %]</textarea>
-                            </td>
-                        </tr>
-                    </table>
+                            <div class="bbag-edit-desc-save">
+                                [% l("Save changes to name or description?") %]
+                                <input type="submit" class="opac-button" value="[% l('Save') %]" />
+                            </div>
                 </form>
             </div>
         </div>
@@ -284,6 +287,18 @@
         <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="post">
         <input type="hidden" name="list" value="[% bbag.id %]" />
         <input type="hidden" name="sort" value="[% CGI.param('sort') | uri %]" />
+        <div class="bbag-content">
+        [% IF bbag.items.size %]
+            <div class="bbag-action">
+                <select name="action" class="bbag-action-field">
+                    <option disabled="disabled" selected="selected">[% l('-- Actions for these items --') %]</option>
+                    <option value="place_hold">[% l('Place hold') %]</option>
+                    <option value="del_item">[% l('Remove from list') %]</option>
+                </select>
+                [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
+                <input class="opac-button" type="submit" value="[% l('Go') %]" />
+            </div>
+        [% END %]
         <table class="bookbag-specific table_no_cell_pad table_no_border_space table_no_border">
             <thead id="acct_list_header">
                 <tr>
@@ -316,15 +331,6 @@
                         | <a href="[% mkurl(ctx.opac_root _ '/myopac/lists', {edit_notes=> bbag.id}) %]">[% l('Edit') %]</a>
                         [% END %]
                     </td>
-                    <td class="list_actions">
-                        <select name="action">
-                            <option disabled="disabled">[% l('-- Actions for these items --') %]</option>
-                            <option value="place_hold">[% l('Place hold') %]</option>
-                            <option value="del_item">[% l('Remove from list') %]</option>
-                        </select>
-                        [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
-                        <input type="submit" value="[% l('Go') %]" />
-                    </td>
                 </tr>
             </thead>
             <tbody>
@@ -350,7 +356,7 @@
                             mkurl(ctx.opac_root _ '/results', {qtype => 'author', query => authorquery}, ['page', 'bbid', 'edit_notes'])
                             -%]">[% attrs.author | html %]</a>
                     </td>
-                    <td class="list_entry">
+                    <td class="list_entry mobile_hide">
                         [% 
                             copy = attrs.holdings.0;
                             IF copy;
@@ -367,21 +373,21 @@
                             END;
                         %]
                     </td>
-                    <td class="list_entry">
+                    <td class="list_entry mobile_hide">
                           [% attrs.pubdate | html %]
                     </td>
-                    <td class="list_entry">
+                    <td class="list_entry mobile_hide">
                           [% attrs.format_label | html %]
                     </td>
                     [% IF CGI.param("edit_notes") == bbag.id %]
-                    <td class="list_entry">
+                    <td class="list_entry mobile_hide">
                         [% FOR note IN item.notes %]
                         <input type="text" name="note-[% note.id %]" value="[% note.note | html %]" />
                         [% END %]
                         <input type="text" name="item-[% item.id %]" />
                     </td>
                     [% ELSE %]
-                    <td class="list_entry">
+                    <td class="list_entry mobile_hide">
                         [% FOR note IN item.notes %]
                         <div>[% note.note | html %]</div>
                         [% END %]
@@ -390,22 +396,22 @@
                 </tr>
                 [% END %]
                 [% IF CGI.param("edit_notes") == bbag.id %]
-                <tr>
-                    <td colspan="3"><!-- All space left of notes column --></td>
+                <tr class="mobile_hide">
+                    <td colspan="6"><!-- All space left of notes column --></td>
                     <td class="save-notes">
                         [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
                         <input type="hidden" name="bbid" value="[% CGI.param('bbid') | html %]" />
-                        <input type="submit" name="save_notes" value="[% l('Save Notes') %]" />
+                        <input type="submit" class="opac-button" name="save_notes" value="[% l('Save Notes') %]" />
                     </td>
                 </tr>
                 [% END %]
             </tbody>
         </table>
-
+        </div>
         [% IF ctx.bb_page_count > 1; %]
             <div class="header_middle" style="padding-top:7px;">
-                <span class="float-left" style="padding-left:34px;">[% l('Navigate Selected List ') %]</span>
-                <span class='float-left' style='padding-left: 10px;'>
+                <div class="bbag-navigate-list">[% l('Navigate Selected List ') %]</div>
+                <div class="bbag-navigate-list-pages">
                     [%- IF item_page > 1 -%]
                         <a href='[% mkurl(ctx.opac_root _ '/myopac/lists', {
                             item_page => item_page - 1
@@ -449,7 +455,7 @@
                             item_page => item_page + 1
                         }) %]'>[% l('Next') %]<span class="nav_arrow_fix">►</span></a>
                     [%- END; -%]
-                 </span>
+                 </div>
             </div>
             <div class="clear-both"></div>
         [% END %]
diff --git a/Open-ILS/src/templates/opac/parts/anon_list.tt2 b/Open-ILS/src/templates/opac/parts/anon_list.tt2
index 2cb06fa..dab23bf 100644
--- a/Open-ILS/src/templates/opac/parts/anon_list.tt2
+++ b/Open-ILS/src/templates/opac/parts/anon_list.tt2
@@ -1,5 +1,7 @@
         [% IF ctx.mylist.size %]
         <div class="bookbag-specific">
+        <p class="big-strong">[% l('Temporary List') %]</p>
+        <div class="sort">
             <form method="get">
                 <label for="anonsort">[% l("Sort list items by: ") %]</label>
                 [% INCLUDE "opac/parts/filtersort.tt2" mode='bookbag'
@@ -7,41 +9,39 @@
                 <input type="hidden" name="id"
                     value="[% CGI.param('id') | html %]" />
                 [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
-                <input type="submit" value="[% l('Sort') %]" />
+                <input class="opac-button" type="submit" value="[% l('Sort') %]" />
             </form>
         </div>
-
         <form action="[% mkurl(ctx.opac_root _ '/mylist/move') %]" method="get">
-        <div>
-            <p class="big-strong">[% l('Temporary List') %]</p>
-            <table class="table_no_border_space table_no_cell_pad table_no_border">
+        <div class="bbag-action" style="clear:both;">
+            <select name="action">
+                <option>[% l('-- Actions for these items --') %]</option>
+                <option value="place_hold">[% l('Place hold') %]</option>
+                <option value="delete">[% l('Remove from list') %]</option>
+                [% IF ctx.user AND ctx.bookbags.size %]
+                    <optgroup label="[% l('Move selected items to list:') %]">
+                    [% FOR bbag IN ctx.bookbags %]]
+                        <option value="[% bbag.id %]" class="selector_actions_for_list_inner_option">[% bbag.name | html %]</option>
+                    [% END %]
+                    </optgroup>
+                [% END %]
+            </select>
+            [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
+            <input class="opac-button" type="submit" value="[% l('Go') %]" />
+        </div>
+        <div class="bbag-content">
+            <table class="bookbag-specific table_no_cell_pad table_no_border_space table_no_border" width="100%">
                 <thead id="acct_list_header_anon">
                     <tr>
-                        <td width="1%" class='padding-left-10'>
+                        <td class='list_checkbox'>
                             <input type="checkbox" onclick="
                                 var inputs=document.getElementsByTagName('input'); 
                                 for (i = 0; i < inputs.length; i++) { 
                                     if (inputs[i].name == 'record' && !inputs[i].disabled) inputs[i].checked = this.checked;}"/>
                         </td>
-                        <td width="40%" class="opac-auto-108"><a href="[% mkurl('', {anonsort=>(CGI.param('anonsort') == 'titlesort' ? 'titlesort.descending' : 'titlesort')}) %]">[% l('Title') %]</a></td>
-                        <td width="40%" class="opac-auto-108"><a href="[% mkurl('', {anonsort=>(CGI.param('anonsort') == 'authorsort' ? 'authorsort.descending' : 'authorsort')}) %]">[% l('Author(s)') %]</a></td>
-                        <td width='18%'>[% l('Local Call Number') %]</td>
-                        <td width="1%" class="nowrap">
-                            <select name="action">
-                                <option>[% l('-- Actions for these items --') %]</option>
-                                <option value="place_hold">[% l('Place hold') %]</option>
-                                <option value="delete">[% l('Remove from list') %]</option>
-                                [% IF ctx.user AND ctx.bookbags.size %]
-                                <optgroup label="[% l('Move selected items to list:') %]">
-                                    [% FOR bbag IN ctx.bookbags %]]
-                                    <option value="[% bbag.id %]" class="selector_actions_for_list_inner_option">[% bbag.name | html %]</option>
-                                    [% END %]
-                                </optgroup>
-                                [% END %]
-                            </select>
-                            [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
-                            <input type="submit" value="[% l('Go') %]" />
-                        </td>
+                        <td><a href="[% mkurl('', {anonsort=>(CGI.param('anonsort') == 'titlesort' ? 'titlesort.descending' : 'titlesort')}) %]">[% l('Title') %]</a></td>
+                        <td><a href="[% mkurl('', {anonsort=>(CGI.param('anonsort') == 'authorsort' ? 'authorsort.descending' : 'authorsort')}) %]">[% l('Author(s)') %]</a></td>
+                        <td>[% l('Local Call Number') %]</td>
                     </tr>
                 </thead>
                 <tbody>
@@ -49,11 +49,11 @@
                         attrs = {marc_xml => ctx.mylist_marc_xml.$item};
                         PROCESS get_marc_attrs args=attrs %]
                     <tr>
-                        <td class="item_list_padding padding-left-10">
+                        <td class="list_checkbox">
                             <input type="checkbox" name="record" value="[% item %]" />
                         </td>
-                        <td class="item_list_padding padding-left-5" ><a href="[% mkurl(ctx.opac_root _ '/record/' _ item, {}, ['edit_notes', 'id']) %]">[% attrs.title | html %]</a></td>
-                        <td class="item_list_padding padding-left-5"><a href="[%- 
+                        <td class="list_entry"><a href="[% mkurl(ctx.opac_root _ '/record/' _ item, {}, ['edit_notes', 'id']) %]">[% attrs.title | html %]</a></td>
+                        <td class="list_entry"><a href="[%- 
                             authorquery = attrs.author | replace('[,\.:;]', '');
                             mkurl(
                                 ctx.opac_root _ '/results',
@@ -61,7 +61,7 @@
                                 ['page', 'id', 'edit_notes']
                             )
                         -%]">[% attrs.author | html %]</a></td>
-                        <td class="item_list_padding padding-left-5">
+                        <td class="list_entry mobile_hide">
                         [% 
                             copy = attrs.holdings.0;
                             IF copy;
@@ -84,4 +84,5 @@
             <br /><br />
         </div>
         </form>
+        </div>
         [% END %]

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

Summary of changes:
 Open-ILS/src/templates/opac/css/style.css.tt2   |  226 ++++++++++++++++++++++-
 Open-ILS/src/templates/opac/myopac/lists.tt2    |  148 ++++++++-------
 Open-ILS/src/templates/opac/parts/anon_list.tt2 |   59 +++---
 3 files changed, 324 insertions(+), 109 deletions(-)


hooks/post-receive
-- 
Evergreen ILS




More information about the open-ils-commits mailing list