From 529d8202cacf5d6889dfb60964307f465298e271 Mon Sep 17 00:00:00 2001
From: Garry Collum <gcollum@gmail.com>
Date: Fri, 29 Jul 2022 22:03:18 +0000
Subject: [PATCH] LP1909583 Bootstrap Opac: Cannot edit title and description

Fixes the editing of the title and description in the Bootstrap Opac lists.
Adds and Edit List button with a collapsible form.

To Test:
1. Create several lists in the Opac.
2. Notice you are unable to edti the title or description.
3. Apply the patch
4. Use the Edit List button to display an editable form for each individual
list.

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
---
 .../opac/css/mediaQuery.css.tt2                    |  2 ++
 .../src/templates-bootstrap/opac/css/style.css.tt2 |  5 ++++
 .../src/templates-bootstrap/opac/myopac/lists.tt2  | 29 +++++++++++++++++++++-
 3 files changed, 35 insertions(+), 1 deletion(-)

diff --git a/Open-ILS/src/templates-bootstrap/opac/css/mediaQuery.css.tt2 b/Open-ILS/src/templates-bootstrap/opac/css/mediaQuery.css.tt2
index e8db387267..d98c372231 100644
--- a/Open-ILS/src/templates-bootstrap/opac/css/mediaQuery.css.tt2
+++ b/Open-ILS/src/templates-bootstrap/opac/css/mediaQuery.css.tt2
@@ -296,6 +296,7 @@ only screen and (max-width: 650px)  {
         margin-bottom:5px;
         padding:5px;
     }
+/*
     #bbag-edit-name {
         [% IF rtl == 't' -%]
         float: right;
@@ -308,6 +309,7 @@ only screen and (max-width: 650px)  {
         width: 220px;
         margin-top:5px;
     }
+*/
     .bbag-content {
         padding:5px;
         border:1px solid [% css_colors.accent_light %];
diff --git a/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2 b/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2
index b774292677..7565e3341a 100755
--- a/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates-bootstrap/opac/css/style.css.tt2
@@ -2539,6 +2539,7 @@ h2.bookbag-name { margin-bottom: 0em; padding-bottom: 0em; }
     font-style: italic;
     max-width: 40em;
 }
+/*
 .bbag-edit-desc-label {
     [% IF rtl == 't' -%]
     float:right;
@@ -2567,6 +2568,8 @@ h2.bookbag-name { margin-bottom: 0em; padding-bottom: 0em; }
     float: left;
     [% END -%]
 }
+*/
+
 .bbag-action {
     [% IF rtl == 't' -%]
     margin-right: 15px;
@@ -3226,6 +3229,7 @@ a.preflib_change {
         margin-bottom:5px;
         padding:5px;
     }
+/*
     #bbag-edit-name {
         [% IF rtl == 't' -%]
         float: right;
@@ -3238,6 +3242,7 @@ a.preflib_change {
         width: 220px;
         margin-top:5px;
     }
+*/
     .bbag-content {
         padding:5px;
         border:1px solid [% css_colors.accent_light %];
diff --git a/Open-ILS/src/templates-bootstrap/opac/myopac/lists.tt2 b/Open-ILS/src/templates-bootstrap/opac/myopac/lists.tt2
index 0cad4832dc..45bba81db1 100755
--- a/Open-ILS/src/templates-bootstrap/opac/myopac/lists.tt2
+++ b/Open-ILS/src/templates-bootstrap/opac/myopac/lists.tt2
@@ -232,11 +232,15 @@
                         {page => '0', bookbag => bbag.id, depth => 0, locg => ctx.search_ou},
                         1
                     )
-                -%]'><i class="fas fa-glasses" aria-hidden="true"></i> [% l('View in Catalogue') %]</a>
+                -%]'><i class="fas fa-glasses" aria-hidden="true"></i> [% l('View in Catalog') %]</a>
 
             </div>
                [% END %]
 
+            <div class="bookbag-controls">
+                <button class="btn btn-action btn-sm m-1" data-toggle="collapse" data-target="#bbag-edit-div-[% bbag.id %]"><i class="fa fa-edit" aria-hidden="true"></i> [% l('Edit List') %]</button>
+            </div>
+
                     </div>
                 <div class="col-3">
                     <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="post" onsubmit="return confirm('[% l('Do you really want to delete this list?') %]')">
@@ -251,6 +255,29 @@
             </div>
         </div>
 
+        <div id="bbag-edit-div-[% bbag.id %]" class="row collapse">
+            <div class="col-12 mt-3 p-2 bg-light">
+            <h5>[% l('Edit List Description') %]</h5>
+            <form method="post">
+                <div class="form-group">
+                    <label for="bbag-edit-name">[% l('Name:') %]</label>
+                    <input class="form-control" name="name" type="text" value="[% bbag.name | html %]" id="bbag-edit-name" />
+                </div>
+                <div class="form-group">
+                <label for="bbag-edit-description">[% l('Description:') %]</label>
+                <textarea name="description" class="form-control"
+                    id="bbag-edit-description">[% bbag.description | html %]
+                </textarea>
+                </div>
+                <input type="hidden" name="bbid" value="[% bbag.id %]" />
+                <input type="hidden" name="action" value="editmeta" />
+                <input type="hidden" name="limit" value="[% limit %]" />
+                <input type="hidden" name="offset" value="[% offset %]" /> 
+                [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
+                <input type="submit" class="btn btn-action btn-sm" value="[% l('Save') %]" />
+            </form>
+            </div>
+        </div>
         
         [% IF CGI.param("bbid") == bbag.id %]
 
-- 
2.11.0