From ddd37e0f5b2b3006b6708a243075eb0c5fabce35 Mon Sep 17 00:00:00 2001
From: Gregoo <versatile.mailbox@gmail.com>
Date: Mon, 27 Jan 2025 18:41:08 +0100
Subject: [PATCH] Fix hide instructions block placement

---
 templates/set/card.html | 36 +++++++++++++++++++-----------------
 1 file changed, 19 insertions(+), 17 deletions(-)

diff --git a/templates/set/card.html b/templates/set/card.html
index d6729ee..e9612eb 100644
--- a/templates/set/card.html
+++ b/templates/set/card.html
@@ -35,26 +35,28 @@
       {% endfor %}
     </ul>
   {% endif %}
-  {% if solo and not config['HIDE_SET_INSTRUCTIONS'] %}
+  {% if solo %}
     <div class="accordion accordion-flush border-top" id="set-details">
       {% if not delete %}
-        {{ accordion.header('Instructions', 'instructions', 'set-details', expanded=open_instructions, quantity=item.instructions | length, icon='file-line', class='p-0') }}
-        <div class="list-group list-group-flush">
-          {% if item.instructions | length %}
-            {% for instruction in item.instructions %}
-            <a class="list-group-item list-group-item-action" href="{{ instruction.url() }}" target="_blank"><i class="ri-arrow-right-long-line"></i> <i class="ri-{{ instruction.icon() }}"></i> {{ instruction.filename }}</a>
-            {% endfor %}
-          {% else %}
-            <span class="list-group-item list-group-item-action text-center"><i class="ri-error-warning-line"></i> No instructions file found.</span>
-            {% if g.login.is_authenticated() %}
-              <a class="list-group-item list-group-item-action" href="{{ url_for('instructions.upload') }}"><i class="ri-upload-line"></i> Upload an instructions file</a>
+        {% if not config['HIDE_SET_INSTRUCTIONS'] %}
+          {{ accordion.header('Instructions', 'instructions', 'set-details', expanded=open_instructions, quantity=item.instructions | length, icon='file-line', class='p-0') }}
+          <div class="list-group list-group-flush">
+            {% if item.instructions | length %}
+              {% for instruction in item.instructions %}
+              <a class="list-group-item list-group-item-action" href="{{ instruction.url() }}" target="_blank"><i class="ri-arrow-right-long-line"></i> <i class="ri-{{ instruction.icon() }}"></i> {{ instruction.filename }}</a>
+              {% endfor %}
+            {% else %}
+              <span class="list-group-item list-group-item-action text-center"><i class="ri-error-warning-line"></i> No instructions file found.</span>
+              {% if g.login.is_authenticated() %}
+                <a class="list-group-item list-group-item-action" href="{{ url_for('instructions.upload') }}"><i class="ri-upload-line"></i> Upload an instructions file</a>
+              {% endif %}
             {% endif %}
-          {% endif %}
-          {% if g.login.is_authenticated() %}
-            <a class="list-group-item list-group-item-action" href="{{ url_for('instructions.download', set=item.fields.set) }}"><i class="ri-download-line"></i> Download instructions from Rebrickable</a>
-          {% endif %}
-        </div>
-        {{ accordion.footer() }}
+            {% if g.login.is_authenticated() %}
+              <a class="list-group-item list-group-item-action" href="{{ url_for('instructions.download', set=item.fields.set) }}"><i class="ri-download-line"></i> Download instructions from Rebrickable</a>
+            {% endif %}
+          </div>
+          {{ accordion.footer() }}
+        {% endif %}
         {{ accordion.table(item.parts(), 'Parts', 'parts-inventory', 'set-details', 'part/table.html', icon='shapes-line')}}
         {% for minifigure in item.minifigures() %}
           {{ accordion.table(minifigure.parts(), minifigure.fields.name, minifigure.fields.figure, 'set-details', 'part/table.html', quantity=minifigure.fields.quantity, icon='group-line', image=minifigure.url_for_image(), alt=minifigure.fields.figure, details=minifigure.url())}}