Only show links to web pages when they are valid (which many seem to

no be at the moment - fallout from the tools.ietf.org move?)

Also move letter icons consistently after the name.

Commit ready for merge.
 - Legacy-Id: 19472
This commit is contained in:
Lars Eggert 2021-10-27 12:01:37 +00:00
parent 63f1df0309
commit 0d1be6093a

View file

@ -29,19 +29,21 @@
<ul class="list-unstyled">
{% for ad in area.ads_and_pre_ads %}
<li>
<a href="mailto:{{ ad.email.address }}"><span class="fa fa-envelope-o tiny"></span></a>
<a href="{% url 'ietf.person.views.profile' email_or_name=ad.person.name %}">{{ ad.person.plain_name }}</a>
<a href="mailto:{{ ad.email.address }}"><span class="fa fa-envelope-o tiny"></span></a>
{% if ad.name == "pre-ad" %} (Incoming AD){% endif %}
</li>
{% endfor %}
</ul>
{% endif %}
{% if area.urls %}
{% if area.urls.is_valid %}
<h3>{{ area.acronym }} area-specific web page{{ area.urls|pluralize}}</h3>
<ul class="list-unstyled">
{% for url in area.urls %}
{% if url.is_valid %}
<li><a href="{{ url.url }}">{{ url.name }}</a></li>
{% endif %}
{% endfor %}
</ul>
{% endif %}
@ -63,8 +65,8 @@
<td><div class="anchor-target" id="{{group.acronym}}"><a href="{% url "ietf.group.views.group_home" group_type=group.type_id acronym=group.acronym %}">{{ group.acronym }}</a></div></td>
<td>
{% if group.ad_role %}
<a href="mailto:{{ group.ad_role.email.address }}"><span class="fa fa-envelope-o tiny"></span></a>
<a href="{% url 'ietf.person.views.profile' email_or_name=group.ad_role.person.name %}">{{ group.ad_role.person.plain_name.split.0 }}</a>
<a href="mailto:{{ group.ad_role.email.address }}"><span class="fa fa-envelope-o tiny"></span></a>
{% endif %}
</td>
<td>{{ group.name }}</td>
@ -97,4 +99,4 @@
{% block js %}
<script src="{% static "jquery.tablesorter/js/jquery.tablesorter.combined.min.js" %}"></script>
{% endblock %}
{% endblock %}