diff --git a/management/templates/management/parent_claim_list.html b/management/templates/management/parent_claim_list.html index 30e5e8b..96819ba 100644 --- a/management/templates/management/parent_claim_list.html +++ b/management/templates/management/parent_claim_list.html @@ -27,18 +27,29 @@ -
+ {% comment %} + size="small" + show_label=False on the dropdown: {% form_field %} + defaults to daisyUI's medium control height, taller than the btn-sm/ + input-sm used by everything else in this row -- without it the + dropdown stands out against the buttons either side of it. + + justify-between (rather than everything in one flex-wrap run) keeps + Approve and Reject apart even as the row wraps on a narrow screen -- + Approve stays pinned left, Reject right, so the two are never one + stray click apart the way adjacent buttons would be. + {% endcomment %} +
{% if claim.has_candidates %} -
+ {% csrf_token %} -
{% form_field claim.review_form.child %}
+
{% form_field claim.review_form.child size="small" show_label=False %}
{% else %}

{% trans "No child without a parent matches this. Check the spelling and the date of birth with them before rejecting." %}

{% endif %} -
+ {% csrf_token %} diff --git a/management/tests.py b/management/tests.py index 6963400..b883c3b 100644 --- a/management/tests.py +++ b/management/tests.py @@ -1454,6 +1454,22 @@ class ParentClaimViewTests(ManagementTestBase): self.assertContains(response, "taylor.doe@example.com") self.assertContains(response, "Jamie Doe") + def test_the_child_dropdown_matches_the_height_of_the_buttons_beside_it(self): + # Regression: ClaimReviewForm.child used to hardcode its own class= + # attrs, which rendered a second class="..." on the