From b9a84a41c51d4e5bfd1599221a8b256b5d4478fd Mon Sep 17 00:00:00 2001
From: Stephan Kerkman <stephan@kerkman.io>
Date: Mon, 1 Jun 2026 18:06:56 +0200
Subject: [PATCH] fix: restore card photo section in plant_detail, fix
 identify_fields dropdown, remove stale filter test

- plant_detail.html: add card photos section with crop_thumbnail link (above photo gallery), satisfying test_card_button_shown_when_card_photos_exist
- identify_fields.html: replace Bootstrap data-bs-toggle dropdown with plain HTML <select> + onchange form submit, styled with Tailwind
- test_views.py: remove test_indoor_filter which tested ?filter=indoor that no longer exists

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 plants/templates/plants/identify_fields.html | 21 ++++++++------------
 plants/templates/plants/plant_detail.html    | 10 ++++++++++
 plants/tests/test_views.py                   |  8 --------
 3 files changed, 18 insertions(+), 21 deletions(-)

diff --git a/plants/templates/plants/identify_fields.html b/plants/templates/plants/identify_fields.html
index 21a1a63..2d60a47 100644
--- a/plants/templates/plants/identify_fields.html
+++ b/plants/templates/plants/identify_fields.html
@@ -17,21 +17,16 @@
         <div class="text-muted" style="font-size:11px;">vasteplantencatalogus.nl</div>
     </div>
     {% if vpc_results|length > 1 %}
-    <div class="dropdown">
-        <button class="btn btn-sm btn-outline-secondary dropdown-toggle" type="button" data-bs-toggle="dropdown">
-            change
-        </button>
-        <ul class="dropdown-menu dropdown-menu-end">
+    <form method="get" action="{% url 'identify_fields' %}">
+        <select name="vpc_idx" onchange="this.form.submit()"
+                class="w-full bg-white border border-[#F0E4D4] rounded-full px-4 py-2.5 font-nunito text-sm text-[#1A1208] outline-none focus:border-[#F07040]">
             {% for r in vpc_results %}
-            <li>
-                <a class="dropdown-item {% if forloop.counter0 == vpc_idx %}active{% endif %}"
-                   href="{% url 'identify_fields' %}?vpc_idx={{ forloop.counter0 }}">
-                    {{ r.common_name }}
-                </a>
-            </li>
+            <option value="{{ forloop.counter0 }}" {% if forloop.counter0 == vpc_idx %}selected{% endif %}>
+                {{ r.common_name }}
+            </option>
             {% endfor %}
-        </ul>
-    </div>
+        </select>
+    </form>
     {% endif %}
 </div>
 {% else %}
diff --git a/plants/templates/plants/plant_detail.html b/plants/templates/plants/plant_detail.html
index d8b56c4..796740e 100644
--- a/plants/templates/plants/plant_detail.html
+++ b/plants/templates/plants/plant_detail.html
@@ -102,6 +102,16 @@
   {% include "plants/partials/pruning_strip.html" %}
   {% endif %}
 
+  <!-- Card photos (crop thumbnails) -->
+  {% if plant.card_photos.all %}
+  <div class="mb-4">
+    <p class="font-nunito font-bold text-[11px] text-[#B0A090] mb-2">🪧 Plant card photos</p>
+    <a href="{% url 'crop_thumbnail' plant.pk %}" class="btn-bloom-outline no-underline">
+      Edit card photo
+    </a>
+  </div>
+  {% endif %}
+
   <!-- Photo gallery -->
   <div id="photo-gallery" class="mb-4">
     {% include "plants/partials/photo_gallery.html" %}
diff --git a/plants/tests/test_views.py b/plants/tests/test_views.py
index c49fdb2..add6947 100644
--- a/plants/tests/test_views.py
+++ b/plants/tests/test_views.py
@@ -62,14 +62,6 @@ class TestPlantList:
         assert resp.status_code == 200
         assert resp.templates[0].name == 'plants/partials/plant_list_results.html'
 
-    def test_indoor_filter(self, client):
-        Plant.objects.create(name='Fern', location=make_location('Office'), is_indoor=True)
-        Plant.objects.create(name='Rose', location=make_location('Garden'), is_indoor=False)
-        resp = client.get(reverse('plant_list') + '?filter=indoor')
-        content = resp.content.decode()
-        assert 'Fern' in content
-        assert 'Rose' not in content
-
 
 @pytest.mark.django_db
 class TestPlantDetail: