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

Author: larseggert

ietf/templates/group/active_wgs.html

@@ -29,19 +29,21 @@ <h3>{{ area.acronym }} Area Director{{ area.ads_and_pre_ads|pluralize }} (AD{{ a
           <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 @@ <h3>{{ area.acronym }} active WG{{ area.groups|pluralize}} ({{area.groups.count}
 		  <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 @@ <h3>No active {{ area.acronym }} WGs</h3>
 
 {% block js %}
 <script src="{% static "jquery.tablesorter/js/jquery.tablesorter.combined.min.js" %}"></script>
-{% endblock %}
+{% endblock %}