Refactor detail templates for condition, finding, and structure to improve layout consistency and readability

This commit is contained in:
Ross
2026-02-02 11:59:35 +00:00
parent 18aef751ca
commit 374ee17911
3 changed files with 108 additions and 108 deletions
+104 -104
View File
@@ -17,141 +17,141 @@
<section class="mb-3"> <section class="mb-3">
<h5 class="mb-2">Details</h5> <h5 class="mb-2">Details</h5>
<dl class="row"> <dl class="row">
<dt class="col-sm-4">Primary name</dt> <dt class="col-sm-4">Primary name</dt>
<dd class="col-sm-8">{% if condition.canonical %}No (alias of <a href="{{ condition.canonical.get_absolute_url }}">{{ condition.canonical.name }}</a>){% else %}Yes{% endif %}</dd> <dd class="col-sm-8">{% if condition.canonical %}No (alias of <a href="{{ condition.canonical.get_absolute_url }}">{{ condition.canonical.name }}</a>){% else %}Yes{% endif %}</dd>
<dt class="col-sm-4">Subspecialty</dt> <dt class="col-sm-4">Subspecialty</dt>
<dd class="col-sm-8">
{% for sub in condition.subspecialty.all %}
<a class="badge bg-dark text-white me-1 text-decoration-none" href="{{ sub.get_absolute_url }}">{{ sub }}</a>
{% empty %}
&mdash;
{% endfor %}
</dd>
<dt class="col-sm-4">Synonyms</dt>
<dd class="col-sm-8">
<div id="synonyms-container">
{% include 'atlas/partials/_synonyms_list.html' %}
</div>
</dd>
<dt class="col-sm-4">Parent</dt>
<dd class="col-sm-8">
{% for parent in condition.parent.all %}
<a class="badge bg-primary text-white text-decoration-none me-1" href="{{ parent.get_absolute_url }}">{{ parent }}</a>
{% empty %}
&mdash;
{% endfor %}
</dd>
<dt class="col-sm-4">Children</dt>
<dd class="col-sm-8">
{% for child in condition.child.all %}
<a class="badge bg-info text-dark text-decoration-none me-1" href="{{ child.get_absolute_url }}">{{ child }}</a>
{% empty %}
&mdash;
{% endfor %}
</dd>
<dt class="col-sm-4">RCR condition</dt>
<dd class="col-sm-8">{% if condition.rcr_curriculum %}Yes{% else %}No{% endif %}</dd>
{% if not condition.rcr_curriculum %}
<dt class="col-sm-4">RCR condition map</dt>
<dd class="col-sm-8"> <dd class="col-sm-8">
{% for sub in condition.subspecialty.all %} {% for map in condition.rcr_curriculum_map.all %}
<a class="badge bg-dark text-white me-1 text-decoration-none" href="{{ sub.get_absolute_url }}">{{ sub }}</a> <a class="badge bg-success text-white text-decoration-none me-1" href="{{ map.get_absolute_url }}">{{ map }}</a>
{% empty %} {% empty %}
&mdash; &mdash;
{% endfor %} {% endfor %}
</dd> </dd>
{% endif %}
<dt class="col-sm-4">Synonyms</dt> </dl>
<dd class="col-sm-8">
<div id="synonyms-container">
{% include 'atlas/partials/_synonyms_list.html' %}
</div>
</dd>
<dt class="col-sm-4">Parent</dt>
<dd class="col-sm-8">
{% for parent in condition.parent.all %}
<a class="badge bg-primary text-white text-decoration-none me-1" href="{{ parent.get_absolute_url }}">{{ parent }}</a>
{% empty %}
&mdash;
{% endfor %}
</dd>
<dt class="col-sm-4">Children</dt>
<dd class="col-sm-8">
{% for child in condition.child.all %}
<a class="badge bg-info text-dark text-decoration-none me-1" href="{{ child.get_absolute_url }}">{{ child }}</a>
{% empty %}
&mdash;
{% endfor %}
</dd>
<dt class="col-sm-4">RCR condition</dt>
<dd class="col-sm-8">{% if condition.rcr_curriculum %}Yes{% else %}No{% endif %}</dd>
{% if not condition.rcr_curriculum %}
<dt class="col-sm-4">RCR condition map</dt>
<dd class="col-sm-8">
{% for map in condition.rcr_curriculum_map.all %}
<a class="badge bg-success text-white text-decoration-none me-1" href="{{ map.get_absolute_url }}">{{ map }}</a>
{% empty %}
&mdash;
{% endfor %}
</dd>
{% endif %}
</dl>
</section> </section>
{% if condition.case_set.exists %} {% if condition.case_set.exists %}
<div class="card mb-3"> <div class="card mb-3">
<div class="card-body"> <div class="card-body">
<h5 class="card-title">Associated Cases</h5> <h5 class="card-title">Associated Cases</h5>
<ul class="list-unstyled mb-0"> <ul class="list-unstyled mb-0">
{% for case in condition.case_set.all %} {% for case in condition.case_set.all %}
<li class="mb-1">{{ case.get_link }}</li> <li class="mb-1">{{ case.get_link }}</li>
{% endfor %} {% endfor %}
</ul> </ul>
</div>
</div> </div>
</div>
{% endif %} {% endif %}
{# Also include cases that reference this condition via CaseDisplaySet #} {# Also include cases that reference this condition via CaseDisplaySet #}
{% if condition.casedisplayset_set.exists %} {% if condition.casedisplayset_set.exists %}
<div class="card mb-3"> <div class="card mb-3">
<div class="card-body"> <div class="card-body">
<h5 class="card-title">Cases via Display Sets</h5> <h5 class="card-title">Cases via Display Sets</h5>
<ul class="list-unstyled mb-0"> <ul class="list-unstyled mb-0">
{% for ds in condition.casedisplayset_set.all %} {% for ds in condition.casedisplayset_set.all %}
<li class="mb-1">{{ ds.case.get_link }} <small class="text-muted">[Display set: {{ ds.name }}]</small></li> <li class="mb-1">{{ ds.case.get_link }} <small class="text-muted">[Display set: {{ ds.name }}]</small></li>
{% endfor %} {% endfor %}
</ul> </ul>
</div>
</div> </div>
</div>
{% endif %} {% endif %}
{% if condition.differential_set.exists %} {% if condition.differential_set.exists %}
<div class="card mb-3"> <div class="card mb-3">
<div class="card-body"> <div class="card-body">
<h5 class="card-title">Associated Case Differentials</h5> <h5 class="card-title">Associated Case Differentials</h5>
<ul class="list-unstyled mb-0"> <ul class="list-unstyled mb-0">
{% for diff in condition.differential_set.all %} {% for diff in condition.differential_set.all %}
<li class="mb-1">{{ diff.case.get_link }} <small class="text-muted">[{{ diff.text }}]</small></li> <li class="mb-1">{{ diff.case.get_link }} <small class="text-muted">[{{ diff.text }}]</small></li>
{% endfor %} {% endfor %}
</ul> </ul>
</div>
</div> </div>
</div>
{% endif %} {% endif %}
{% if condition.sbas_questions.exists %} {% if condition.sbas_questions.exists %}
<div class="card mb-3"> <div class="card mb-3">
<div class="card-body"> <div class="card-body">
<h5 class="card-title">Related SBA Questions</h5> <h5 class="card-title">Related SBA Questions</h5>
<ul class="list-unstyled mb-0"> <ul class="list-unstyled mb-0">
{% for q in condition.sbas_questions.all %} {% for q in condition.sbas_questions.all %}
<li class="mb-1"><a href="{{ q.get_absolute_url }}">{% if q.title %}{{ q.title }}{% else %}{{ q.get_stem_stripped|truncatechars:140 }}{% endif %}</a></li> <li class="mb-1"><a href="{{ q.get_absolute_url }}">{% if q.title %}{{ q.title }}{% else %}{{ q.get_stem_stripped|truncatechars:140 }}{% endif %}</a></li>
{% endfor %} {% endfor %}
</ul> </ul>
</div>
</div> </div>
</div>
{% endif %} {% endif %}
{% if not condition.rcr_curriculum and can_merge %} {% if not condition.rcr_curriculum and can_merge %}
<div class="card mb-3"> <div class="card mb-3">
<div class="card-body"> <div class="card-body">
<h5 class="card-title">Merge</h5> <h5 class="card-title">Merge</h5>
<p class="small text-muted">Merge this condition into another. This will transfer associated cases and differentials but will not transfer synonyms, parents or children.</p> <p class="small text-muted">Merge this condition into another. This will transfer associated cases and differentials but will not transfer synonyms, parents or children.</p>
<p> <p>
<a class="btn btn-sm btn-outline-primary" data-bs-toggle="collapse" href="#mergeCollapse" role="button" aria-expanded="false" aria-controls="mergeCollapse">Show merge form</a> <a class="btn btn-sm btn-outline-primary" data-bs-toggle="collapse" href="#mergeCollapse" role="button" aria-expanded="false" aria-controls="mergeCollapse">Show merge form</a>
</p> </p>
<div class="collapse" id="mergeCollapse"> <div class="collapse" id="mergeCollapse">
<div class="card card-body"> <div class="card card-body">
<form method="POST" id="condition-merge-form"> <form method="POST" id="condition-merge-form">
{% csrf_token %} {% csrf_token %}
<div class="mb-3"> <div class="mb-3">
{{ form.as_p }} {{ form.as_p }}
</div> </div>
<button class="btn btn-danger" <button class="btn btn-danger"
hx-post='{% url "atlas:condition_merge" condition.pk %}' hx-post='{% url "atlas:condition_merge" condition.pk %}'
hx-target="#merge-options" hx-swap="innerHTML" hx-target="#merge-options" hx-swap="innerHTML"
hx-confirm="This will merge {{condition.name}} into the selected condition, this cannot be undone. Continue?"> hx-confirm="This will merge {{condition.name}} into the selected condition, this cannot be undone. Continue?">
Merge Merge
</button> </button>
</form> </form>
<div id="merge-options" class="mt-3"></div> <div id="merge-options" class="mt-3"></div>
</div>
</div> </div>
</div> </div>
</div> </div>
</div>
{% endif %} {% endif %}
</div> </div>
+2 -2
View File
@@ -12,8 +12,8 @@
</div> </div>
<div> <div>
<h3>Name: {{finding.name}}</h3> <h3>Name: {{finding.name}}</h3>
Primary name: {% if finding.canonical %}No (alias of <a href="{{ finding.canonical.get_absolute_url }}">{{ finding.canonical.name }}</a>){% else %}Yes{% endif %}<br /> Primary name: {% if finding.canonical %}No (alias of <a href="{{ finding.canonical.get_absolute_url }}">{{ finding.canonical.name }}</a>){% else %}Yes{% endif %}<br />
Synonyms: {{ finding.get_synonym_link|safe }}<br /> Synonyms: {{ finding.get_synonym_link|safe }}<br />
</div> </div>
{% if finding.seriesfinding_set.all %} {% if finding.seriesfinding_set.all %}
<h4>Associated Cases</h4> <h4>Associated Cases</h4>
+2 -2
View File
@@ -14,8 +14,8 @@
</div> </div>
<div> <div>
<h3>Name: {{structure.name}}</h3> <h3>Name: {{structure.name}}</h3>
Primary name: {% if structure.canonical %}No (alias of <a href="{{ structure.canonical.get_absolute_url }}">{{ structure.canonical.name }}</a>){% else %}Yes{% endif %}<br /> Primary name: {% if structure.canonical %}No (alias of <a href="{{ structure.canonical.get_absolute_url }}">{{ structure.canonical.name }}</a>){% else %}Yes{% endif %}<br />
Synonyms: {{ structure.get_synonym_link|safe }}<br /> Synonyms: {{ structure.get_synonym_link|safe }}<br />
</div> </div>
<h4>Associated Cases</h4> <h4>Associated Cases</h4>
<ul> <ul>