Skip to content

Commit dafc424

Browse files
Review medical information tweaks
- Reorder Medical history & Symptoms panels - Add description text to Lump form - Remove Other (symptom) form hint text - Remove horizontal rules from panels when multiple symptoms are present - Update back link text on lump form
1 parent 3614507 commit dafc424

File tree

5 files changed

+26
-9
lines changed

5 files changed

+26
-9
lines changed

manage_breast_screening/mammograms/jinja2/mammograms/medical_information/section_cards.jinja

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@
5252
"html": insetHtml,
5353
"classes": "nhsuk-u-margin-top-0 nhsuk-u-margin-bottom-4"
5454
}) }}
55+
<hr>
5556
{% endif %}
5657

5758
{% if not read_only %}
58-
<hr>
5959
<h3>Add symptoms</h3>
6060

6161
<div class="nhsuk-button-group app-button-group--small">

manage_breast_screening/mammograms/jinja2/mammograms/medical_information/symptoms/forms/inline_area_radios.jinja

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,9 +69,6 @@
6969
{
7070
"value": "OTHER",
7171
"text": "Other",
72-
"hint": {
73-
"text": "For example, the armpit"
74-
},
7572
"conditional": {
7673
"html": form.area_description_other.as_field_group()
7774
}

manage_breast_screening/mammograms/jinja2/mammograms/medical_information/symptoms/forms/simple_symptom.jinja

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,12 @@
33
{% from "nhsuk/components/fieldset/macro.jinja" import fieldset %}
44

55
{% block form %}
6+
{% if heading_description %}
7+
<div class="nhsuk-inset-text nhsuk-u-margin-top-0 nhsuk-u-margin-bottom-4">
8+
<p>{{ heading_description }}</p>
9+
</div>
10+
{% endif %}
11+
612
{% include "mammograms/medical_information/symptoms/forms/inline_area_radios.jinja" %}
713

814
{% do form.when_started.add_divider_after("OVER_THREE_YEARS", "or") %}

manage_breast_screening/mammograms/jinja2/mammograms/record_medical_information.jinja

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,17 +29,17 @@
2929
{% endcall %}
3030

3131
{% call card_with_status(
32-
presented_section=presenter.get_section(sections.SYMPTOMS),
32+
presented_section=presenter.get_section(sections.MEDICAL_HISTORY),
3333
csrf_input=csrf_input,
3434
) %}
35-
{{ symptoms_content(presenter) }}
35+
{{ medical_history_content(presenter) }}
3636
{% endcall %}
3737

3838
{% call card_with_status(
39-
presented_section=presenter.get_section(sections.MEDICAL_HISTORY),
39+
presented_section=presenter.get_section(sections.SYMPTOMS),
4040
csrf_input=csrf_input,
4141
) %}
42-
{{ medical_history_content(presenter) }}
42+
{{ symptoms_content(presenter) }}
4343
{% endcall %}
4444

4545
{% call card_with_status(

manage_breast_screening/mammograms/views/symptom_views.py

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ def get_back_link_params(self):
3737
"mammograms:record_medical_information",
3838
kwargs={"pk": self.appointment_pk},
3939
),
40-
"text": "Back to appointment",
40+
"text": "Back",
4141
}
4242

4343
def get_context_data(self, **kwargs):
@@ -130,6 +130,13 @@ class AddLumpView(AddSymptomView):
130130
form_class = LumpForm
131131
template_name = "mammograms/medical_information/symptoms/forms/simple_symptom.jinja"
132132

133+
def get_context_data(self, **kwargs):
134+
context = super().get_context_data(**kwargs)
135+
context["heading_description"] = (
136+
"Lumps are a recognised symptom of breast cancer. Information recorded here will be highlighted during image reading."
137+
)
138+
return context
139+
133140

134141
class AddSwellingOrShapeChangeView(AddSymptomView):
135142
"""
@@ -188,6 +195,13 @@ class ChangeLumpView(ChangeSymptomView):
188195
def extra_filters(self):
189196
return {"symptom_type_id": SymptomType.LUMP}
190197

198+
def get_context_data(self, **kwargs):
199+
context = super().get_context_data(**kwargs)
200+
context["heading_description"] = (
201+
"Record whether the lump is in the left breast, right breast or both."
202+
)
203+
return context
204+
191205

192206
class ChangeSwellingOrShapeChangeView(ChangeSymptomView):
193207
"""

0 commit comments

Comments
 (0)