From 26349c98d4c35fa79efe04ab599bf0cbab3bd98c Mon Sep 17 00:00:00 2001
From: Damien De Paoli <ddp@depaoli.id.au>
Date: Tue, 4 Jul 2023 23:54:32 +1000
Subject: [PATCH] added edit author button/link, and fixed broken hover since I
 changed to bals from author.   Also removed the old author code, finally made
 it so when you hover over remove button we now also highlight the edit button
 for the author to be red as well.  Hovering over edit link higlights the
 relevant author blue to let you know which one you will go to

---
 templates/book.html | 61 +++++++++++++++------------------------------
 1 file changed, 20 insertions(+), 41 deletions(-)

diff --git a/templates/book.html b/templates/book.html
index be06915..24a5f5c 100644
--- a/templates/book.html
+++ b/templates/book.html
@@ -200,10 +200,6 @@ function AddAuthorToBook(num) {
 }
 </script>
 
-{#
-{% set keys = [ 'title', 'bals', 'author', 'publisher', 'genre', 'owned', 'covertype', 'condition', 'year_published', 'rating', 'notes', 'blurb' ] %}
-#}
-
 {% set keys = [ 'title', 'bals', 'publisher', 'genre', 'owned', 'covertype', 'condition', 'year_published', 'rating', 'notes', 'blurb' ] %}
 <div class="container-fluid">
 	<div class="form-row"><h3 class="offset-2">{{page_title}}</h3></div>
@@ -257,6 +253,11 @@ function AddAuthorToBook(num) {
                                     </svg>
                                 </button>
                             {% endif %}
+                            <button id="author-edit-{{cnt.idx}}" class="btn" type="button" onclick="window.location.href='/author/{{a.author.id}}'">
+                                    <svg width="22" height="22" fill="currentColor">
+                                        <use xlink:href="{{url_for('static', filename='icons.svg')}}#edit"/>
+                                    </svg>
+                            </button>
                             <select class="form-select input-group-append" name="author-{{cnt.idx}}" id="author-{{cnt.idx}}">
                             {% for auth in author_list %}
                                 {% set aname=auth.surname+", "+auth.firstnames %}
@@ -283,39 +284,6 @@ function AddAuthorToBook(num) {
                         {% endif %}
                         <svg width="22" height="22" fill="currentColor"><use xlink:href="{{url_for('static', filename='icons.svg')}}#plus"/></svg>
                         </button>
-					{% elif key == "author" %}
-							{% set cnt = namespace(idx=0) %}
-							{% for objects in books[key] %}
-									{% if cnt.idx > 0 %}
-										<button id="author-rem-{{cnt.idx}}" class="btn btn-outline-danger input-group-append" type="button" onClick="RemoveAuthorFromBook({{cnt.idx}})">
-											<svg width="22" height="22" fill="currentColor">
-												<use xlink:href="{{url_for('static', filename='icons.svg')}}#minus"/>
-											</svg>
-										</button>
-									{% endif %}
-									<select class="form-select input-group-append" name="author-{{cnt.idx}}" id="author-{{cnt.idx}}">
-									{% for auth in author_list %}
-										{% set aname=auth.surname+", "+auth.firstnames %}
-										<option value="{{auth.id}}"
-										{% if books.author[cnt.idx].id == auth.id %}
-											selected
-										{% endif %}
-										>{{aname}}</option>
-									{% endfor %}
-									</select>
-								{% set cnt.idx = cnt.idx+1 %}
-							{% endfor %}
-							{% if cnt.idx == 0 %}
-									<select class="form-select input-group-append" name="author-{{cnt.idx}}" id="author-{{cnt.idx}}">
-										{% for auth in author_list %}
-											{% set aname=auth.surname+", "+auth.firstnames %}
-											<option value="{{auth.id}}">{{aname}}</option>
-										{% endfor %}
-									</select>
-							{% endif %}
-								<button id="author-plus" class="btn btn-outline-success input-group-append" type="button" onClick="AddAuthorToBook({{cnt.idx}})">
-								<svg width="22" height="22" fill="currentColor"><use xlink:href="{{url_for('static', filename='icons.svg')}}#plus"/></svg>
-								</button>
 					{% else %}
 							{% set rows=4 %}
 							{% if key == "notes" %}
@@ -459,6 +427,7 @@ function AddAuthorToBook(num) {
 
 {% endblock main_content %}
 {% if books.child_ref|length %}
+{% endif %}
 	{% block script_content %}
 		<script>
 			$(document).ready( function() {
@@ -474,14 +443,24 @@ function AddAuthorToBook(num) {
 					{% endif %}
 				{% endif %}
 				{% set cnt = namespace(idx=1) %}
-				{% for authors in books['author'] %}
+				{% for authors in books['bals'] %}
 					$('#author-rem-{{cnt.idx}}').hover( 
-						function() { $('#author-{{cnt.idx}}').prop( 'style', 'color:red' ) }, 
-						function() { $('#author-{{cnt.idx}}').prop( 'style', '' ) } )
+						function() { 
+                            $('#author-{{cnt.idx}}').prop( 'style', 'color:red' )
+                            $('#author-edit-{{cnt.idx}}').prop(  'style', 'color:red' )
+                        }, 
+						function() { 
+                            $('#author-{{cnt.idx}}').prop( 'style', '' ) 
+                            $('#author-edit-{{cnt.idx}}').prop( 'style', '' ) 
+                        }
+                    )
+                    $('#author-edit-{{cnt.idx}}').hover(
+						function() { $('#author-{{cnt.idx}}').prop( 'style', 'color:blue' ) },
+						function() { $('#author-{{cnt.idx}}').prop( 'style', '' ) }
+                    )
 					{% set cnt.idx = cnt.idx+1 %}
 				{% endfor %}
                 $('#owned').click( function() { if( $("#owned option:selected").text() == 'On Wish List' ) { $('#covertype').val( $('#covertype option:last').val() ); $('#condition').val( $('#condition option:last').val() ); $('#year_published').val(''); $('#rating').val( $('#rating option:last').val() ) } } )
 			} )
 		</script>
 	{% endblock script_content %}
-{% endif %}