From cc1c8ed7854603fc1d3ed35497b1827f2ac35101 Mon Sep 17 00:00:00 2001
From: jules <zeroth@free.fr>
Date: Thu, 19 Nov 2020 17:11:28 +0100
Subject: [PATCH] artwork template divs are fixed and annotated

---
 templates/artwork.html | 83 ++++++++++++++++++++++++++----------------
 1 file changed, 52 insertions(+), 31 deletions(-)

diff --git a/templates/artwork.html b/templates/artwork.html
index a7d0eef..67d7348 100644
--- a/templates/artwork.html
+++ b/templates/artwork.html
@@ -46,17 +46,19 @@
 						<img class="artwork-about-imgs" src="{{ url_for('static', filename='/imgs/Icons/placeholder_no-image2.png') }}">
 					{% else %}
 						{% for x in artworkimages['results']['bindings'] %}
-							{% if loop.index <= 1 %}
+							
 								{% if "image" in x %}
 								<img class="artwork-about-imgs" src='{{ x["image"]["value"] | replace("wiki/File:","wiki/Special:Redirect/file/") }}'>
 								{% endif %}
-							{% endif %}
+							
 						{% endfor %}
 					{% endif %}
 				</div>
-				<!-- other stuff like downloads, absent for now -->
 
-				<!-- DOWNLOADS AND LINKS -->
+<!-- other stuff like downloads, absent for now -->
+				<div class="artwork-about-under-img">
+
+<!-- DOWNLOADS AND LINKS -->
 				<div>
 					{% if artworklinksanddownloads['results']['bindings']==[] %}
 					{% else %}
@@ -93,13 +95,13 @@
 					{% endif %}
 				</div>
 				
-				<!-- OTHER DIGITAL ARTEFACTS -->
+<!-- OTHER DIGITAL ARTEFACTS -->
 				<div>
 					{% if artworkartefacts['results']['bindings']==[] %}
 					<p>information not available</p>
 					{% else %}
 					<p class="label-item">OTHER DIGTAL ARTEFACTS</p>
-					<!-- video images audio -->
+<!-- video images audio -->
 						{% for x in artworkartefacts['results']['bindings'] %}
 
 						{% if ('VideoFile' not in x) and ('AudioFile' not in x) and ('ImageFile' not in x) %}
@@ -131,7 +133,7 @@
 					{% endif %}
 				</div>
 
-				<!-- DISTRIBUTOR LINKS -->
+<!-- DISTRIBUTOR LINKS -->
 				<div>
 					{% if artworkdistributorlinks['results']['bindings']==[] %}
 					{% else %}		
@@ -149,19 +151,20 @@
 
 							{% endif %}
 						{% endfor %}	
-						</p>
 						
 					{% endif %}
 				</div>
-				</div>
-
-		</div>
+			</div>
+</div>
+	
+<!-- end of other stuff like downloads, absent for now -->
 
-		<!-- RIGHT COLUMN -->
-		<div class="artwork-rightcolumn"> 
-			<!-- RIGHT TOP -->
+<!-- RIGHT COLUMN -->
+<div class="artwork-rightcolumn"> 
 
-				<!-- contributors -->
+<!-- RIGHT TOP -->
+<div class="artwork-rightcolumn-top">
+<!-- contributors -->
 				<div class="artwork-rightcolumn-top-contributors">
 						<p class="label-item">CONTRIBUTORS</p>
 						<!-- check if info not available -->
@@ -182,7 +185,7 @@
 						{% endif %}
 				</div>
 
-				<!-- date -->
+<!-- date -->
 				<div class="artwork-rightcolumn-top-date">
 						<p class="label-item">DATE</p>
 							<!-- check if info not available -->
@@ -203,9 +206,11 @@
 								{% endif %}
 							{% endfor %}
 							{% endif %}
+					</div>
+<!-- end of date -->
 
-				<!-- publishers removed div .artwork-rightcolumn-top-publishers-->
-
+<!-- publishers-->
+				<div class="artwork-rightcolumn-top-publishers">
 						<p class="label-item">PUBLISHERS</p>
 						<!-- loop through publishers -->
 						{% if artworkpublisher['results']['bindings']==[] %}
@@ -222,17 +227,21 @@
 							{% if 'role' in x %}
 								<span>{{ x['role']['value'] }}</span>
 							{% endif %}
-							<div>
+							</div>
 						{% endfor %}
 
 						{% endif %}
+			
 				</div>
-				</div>
-				</div>
+<!-- end of publishers-->
+
+</div>
+<!-- end of about right top -->
 
+<!-- start of about right column bottom -->
+			<div class="artwork-rightcolumn-bottom">
 
-			<div class="artwork-about-bottom">
-				<!-- description -->
+<!-- description -->
 
 				<!-- if there is a description add description here -->
 				{% if artworkdescriptiondata['results']['bindings']==[] %}
@@ -273,9 +282,9 @@
 				</div>
 				{% endfor %}
 				{% endif %}
-				<!-- end description loop -->
+<!-- end description loop -->
 
-				<!-- exhibition and distribution history loop-->
+<!-- exhibition and distribution history loop-->
 				<!-- if exhibition in, then do show that -->
 				{% if artworkexhibitiondata['results']['bindings']==[] %}
 				{% else %}
@@ -285,7 +294,7 @@
 					<p class="label-item">EXHIBITION AND DISTRIBUTION HISTORY</p>
 
 					<div>
-						{{ artworkexhibitiontext | safe }}
+						{{ x['text'] | safe }}
 					</div>
 					<!-- right ride with author etc LATER-->
 					<div>
@@ -311,16 +320,26 @@
 				</div>
 				{% endfor %}
 				{% endif %}
-				<!-- end exhibition loop -->
+<!-- end exhibition loop -->
 
-				<!-- end of right column -->
+<!-- end of right column -->
 			</div>
+
 <!-- end of about -->
 	</div>
 
-	<!-- lists DESIGN IF NO LISTS TO BE CLARIFIE-->
-	<div class="artwork-nav-label">LISTS</div>
+
+
+
+
+
+</div>
+<!-- END OF ABOUT -->
+
+<!-- LISTS DESIGN IF NO LISTS TO BE CLARIFIED-->
 	<div class="artwork-lists">
+		<div class="artwork-nav-label">LISTS</div>
+
 		{% if artworklists['results']['bindings']==[] %}
 			<p>The artwork isn't featured in any lists yet</p>
 		{% else %}
@@ -347,6 +366,7 @@
 		{% endfor %}
 		{% endif %}
 	</div>
+<!-- END OF LISTS -->
 
 
 <!-- copies in collections -->
@@ -377,6 +397,7 @@
 			</div>
 		{% endfor %}
 	</div>
+<!-- end copies in collections -->
 
 <!-- Related works -->
 	<div class="artwork-relatedwrapper">
@@ -411,7 +432,7 @@
 			</div>
 		{% endfor %}
 	</div>
+<!-- End related works -->
 
 </div>
-
 {% endblock content %}
\ No newline at end of file