updated and cleaned up the papers logic

Author: moneuron

_includes/header.html

@@ -15,7 +15,7 @@ <h1>{{ site.title | escape }}</h1>
               {%- assign resolved_href = href | relative_url -%}
             {%- endif -%}
           <li>
-            <a href="{{ resolved_href }}"{%- if item.id -%} id="{{ item.id }}"{%- endif -%}{%- if item.id == 'contact-toggle' -%} aria-controls="email-blockquote" aria-expanded="false"{%- endif -%}>{{ item.label }}</a>
+            <a href="{{ resolved_href }}" {%- if item.id %} id="{{ item.id }}"{% endif -%} {%- if item.id == 'contact-toggle' %} aria-controls="email-blockquote" aria-expanded="false"{% endif -%}>{{ item.label }}</a>
           </li>
           {%- endfor -%}
         </ul>

_includes/papers.html

@@ -1,27 +1,48 @@
 <section aria-label="Papers">
-  {% for paper in site.data.papers %}
+{%- for paper in site.data.papers -%}
   <article class="paper-item">
-    · {% assign authors_count = paper.authors | size %}
-    {% if authors_count > 0 %}
-      {% for author in paper.authors %}
-        {% if author.highlight %}<strong>{{ author.name }}</strong>{% else %}{{ author.name }}{% endif %}{% unless forloop.last %}, {% endunless %}
-      {% endfor %}. 
-    {% endif %}
-
-    {% assign title = paper.title | strip %}
-  {% assign title_last = title | slice: -1 %}
-  {{ title }}{% unless title_last == '.' or title_last == '?' or title_last == '!' %}.{% endunless %} <em>{{ paper.venue }}</em>, {{ paper.year }}.
-
-    {% assign links_count = paper.links | size %}
-    {% if links_count > 0 %} <sub><strong>
-      {% for link in paper.links %}
-        <a href="{{ link.url }}">{{ link.label }}</a>{% unless forloop.last %} · {% endunless %}
-      {% endfor %}
-    </strong></sub>{% endif %}
-
-    {% if paper.doi %} <sub class="citation"{% if paper.citation_id %} id="{{ paper.citation_id }}"{% endif %} data-doi="{{ paper.doi }}"></sub>{% endif %}
-
-    <sub><strong><code>{{ paper.type }}</code></strong></sub>
+    ·
+    {%- assign authors_count = paper.authors | size -%}
+
+    {%- if authors_count > 0 -%}
+      {%- for author in paper.authors -%}
+        {%- if author.highlight -%}
+          <strong>{{- author.name -}}</strong>
+        {%- else -%}
+          {{- author.name -}}
+        {%- endif -%}
+        {%- unless forloop.last -%}, {% endunless -%}
+      {%- endfor -%}.
+    {%- endif -%}
+
+    {%- assign title = paper.title | strip -%}
+    {%- assign title_last = title | slice: -1 -%}
+
+    {{- title -}}
+    {%- unless title_last == '.' or title_last == '?' or title_last == '!' -%}.{%- endunless -%}
+
+    <em>{{- paper.venue -}}</em>, {{- paper.year -}}.
+
+    {%- assign links_count = paper.links | size -%}
+
+    {%- if links_count > 0 -%}
+      <sub><strong>
+        {%- for link in paper.links -%}
+          <a href="{{ link.url }}">{{- link.label -}}</a>
+          {%- unless forloop.last -%} · {% endunless -%}
+        {%- endfor -%}
+      </strong></sub>
+    {%- endif -%}
+
+    {%- if paper.doi -%}
+      <sub
+        class="citation"
+        {%- if paper.citation_id -%} id="{{ paper.citation_id }}"{%- endif -%}
+        data-doi="{{ paper.doi }}">
+      </sub>
+    {%- endif -%}
+
+    <sub><strong><code>{{- paper.type -}}</code></strong></sub>
   </article>
-  {% endfor %}
-</section>
+{%- endfor -%}
+</section>

assets/script.js

@@ -35,23 +35,11 @@ document.addEventListener("DOMContentLoaded", () => {
       }
     });
 
-    // Update total + h-index (prefer precomputed values from citations.json)
+    // Update total + h-index
     if (totalDisplay) {
       const totalFromJson = Number(data._total_citations);
       const hIndexFromJson = Number(data._h_index);
-
-      if (Number.isFinite(totalFromJson) && Number.isFinite(hIndexFromJson)) {
-        totalDisplay.textContent = `‒ CITATIONS: ${totalFromJson} · H-INDEX: ${hIndexFromJson} ‒`;
-      } else if (citationCounts.length) {
-        const sorted = citationCounts.slice().sort((a, b) => b - a);
-        let hIndex = 0;
-        for (let i = 0; i < sorted.length; i++) {
-          if (sorted[i] >= i + 1) hIndex = i + 1;
-          else break;
-        }
-        const total = citationCounts.reduce((a, b) => a + b, 0);
-        totalDisplay.textContent = `‒ CITATIONS: ${total} · H-INDEX: ${hIndex} ‒`;
-      }
+      totalDisplay.textContent = `‒ CITATIONS: ${totalFromJson} · H-INDEX: ${hIndexFromJson} ‒`;
     }
 
     const yearsData = data._yearly_totals;