Updates following review

Author: Tim Davies

prefix_finder/frontend/templates/results.html

@@ -5,30 +5,30 @@
 {% for result in results %}
   <div class="col-xs-12 col-md-6">
   <div class="panel panel-default">
-    <div class="panel-heading"><span class="badge pull-right" data-toggle="popover" data-placement="bottom" title="Quality score" data-html="true" data-content="{%for key,value in result.quality_explained.items %}{{ key }} - {{ value }}<br/>{%endfor%}" data-trigger="hover">Quality score: {{ result.quality }}</span>
-      <h4><a href="{% url 'list' result.code %}" data-toggle="tooltip" data-placement="top" title="More info">{{result.name.en}} ({{result.code}})</a> </h4>
+    <div class="panel-heading"><span class="badge pull-right" data-toggle="popover" data-placement="bottom" title="Quality score" data-html="true" data-content="{% for key,value in result.quality_explained.items %}{{ key }} - {{ value }}<br/>{% endfor %} data-trigger="hover">Quality score: {{ result.quality }}</span>
+      <h4><a href="{% url 'list' result.code %}" data-toggle="tooltip" data-placement="top" title="More info">{{ result.name.en }} ({{ result.code }})</a> </h4>
 
     </div>
     <div class="panel-body">
     <p style="font-size:larger;">
-    Use the list code <code>{{result.code}}</code> and an identifier found at <a href="{{result.url}}" target="_blank">{{result.url|truncatechars:100}}</a>
+    Use the list code <code>{{ result.code }}</code> and an identifier found at <a href="{{ result.url }}" target="_blank">{{ result.url|truncatechars:100 }}</a>
     </p>
     <p>
-    {{result.description.en|truncatewords_html:50}} <a href="{% url 'list' result.code %}">Read more</a>
+    {{ result.description.en|truncatewords_html:50 }} <a href="{% url 'list' result.code %}">Read more</a>
     </p>
     <p>
     This is a <strong>{{ result.listType }}</strong> list.  
     {% if result.listType = "primary" %} 
       Organisations are generally entered into this list when they are registered 
-        {%if result.structure %} as a {{ result.structure_titles|join_or:True }} {% endif %}
+        {% if result.structure %} as a {{ result.structure_titles|join_or:True }} {% endif %}
         {% if result.subnationalCoverage %}   
           in {{ result.subnationalCoverage_titles|join_or }}.
         {% else %} 
           in {{ result.coverage_titles|join_or }}.
         {% endif %}
     {% endif %}
     {% if result.listType = "secondary" %} 
-      {% if result.structure %}{{ result.structure_titles|join_or }} organisations{% else %}{% if result.sector %}{{ result.sector_titles|join_or }} organisations{%else%} Organisations{% endif %}{% endif %} may be entered onto this offical list as a result of a transaction with the list owner. 
+      {% if result.structure %}{{ result.structure_titles|join_or }} organisations{% else %}{% if result.sector %}{{ result.sector_titles|join_or }} organisations{% else %} Organisations{% endif %}{% endif %} may be entered onto this offical list as a result of a transaction with the list owner. 
     {% endif %}
     {% if result.listType = "third_party" %} 
       This list is maintained by a third-party who may or may not have had interactions or transactions with the organiastions listed.
@@ -46,15 +46,15 @@ <h4><a href="{% url 'list' result.code %}" data-toggle="tooltip" data-placement=
     {% if result.sector %}<p>
     It includes organisations working in the following sectors: {{ result.sector_titles|join_or }}
     </p>{% endif %}
-    <a class="pull-right" href="{% url 'list' result.code %}"><button class="btn">More info</button></a>
+    <a class="pull-right" href="{% url 'list' result.code %}"><button class="btn" style="margin-top:25px;">More info</button></a>
     <hr/>
     <p>
       <strong>Example identifiers</strong>: {{ result.access.exampleIdentifiers }}
     </p>
     {% if local %}
     <strong>Debug info</strong>
     {{ result.relevance }} | {{ result.relevance_debug }}  
-    {%endif%}
+    {% endif %}
     </div>
   </div>
   </div>

prefix_finder/frontend/views.py

@@ -16,7 +16,7 @@
     "MATCH_DROPDOWN": 10,
     "MATCH_DROPDOWN_ONLY_VALUE": 10,
     "MATCH_EMPTY": 2,
-    "RECOMENDED_RELEVANCE_THRESHOLD": 5,
+    "RECOMMENDED_RELEVANCE_THRESHOLD": 5,
     "SUGGESTED_RELEVANCE_THRESHOLD": 35,
     "SUGGESTED_QUALITY_THRESHOLD": 45
 }
@@ -323,7 +323,7 @@ def filter_and_score_results(query):
             and value['quality'] > RELEVANCE["SUGGESTED_QUALITY_THRESHOLD"]
             and not all_results['suggested'] or (all_results['suggested'] and value['relevance'] == all_results['suggested'][0]['relevance'])):
             all_results['suggested'].append(value)
-        elif value['relevance'] >= RELEVANCE["RECOMENDED_RELEVANCE_THRESHOLD"]:
+        elif value['relevance'] >= RELEVANCE["RECOMMENDED_RELEVANCE_THRESHOLD"]:
             all_results['recommended'].append(value)
         else:
             all_results['other'].append(value)