From 39973eed99fc4b2768bb561b7216b441e76ffcf9 Mon Sep 17 00:00:00 2001
From: Nick Young <nick.young@auckland.ac.nz>
Date: Tue, 31 Jan 2017 17:10:48 +1300
Subject: [PATCH 1/2] ensure dt elements have an id set

---
 _includes/javascript.html | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/_includes/javascript.html b/_includes/javascript.html
index a2066c2..d0d2249 100644
--- a/_includes/javascript.html
+++ b/_includes/javascript.html
@@ -12,3 +12,16 @@
   ga('create', 'UA-37305346-2', 'auto');
   ga('send', 'pageview');
 </script>
+<script>
+  $('dt').each(function () {
+    if (!this.id) {
+      var id = $(this).text();
+      var index = id.indexOf('(');
+      if (index > 0) {
+        id = id.substring(0, index);
+      }
+      id = id.trim().replace(/ /g, '-').toLowerCase();
+      this.id = id;
+    }
+  });
+</script>
\ No newline at end of file

From 0cb3ca5b6814c5ee5dfc4e46d0256a9b2d2e0ada Mon Sep 17 00:00:00 2001
From: Nick Young <nbay92@gmail.com>
Date: Wed, 1 Feb 2017 10:19:36 +1300
Subject: [PATCH 2/2] Add javascript comments to the dt-ids snippet

---
 _includes/javascript.html | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/_includes/javascript.html b/_includes/javascript.html
index d0d2249..93b5152 100644
--- a/_includes/javascript.html
+++ b/_includes/javascript.html
@@ -13,15 +13,21 @@
   ga('send', 'pageview');
 </script>
 <script>
+  // This snippet fixes a bug caused by Github's pages-gem using kramdown v1.11.1.
+  // In order for anchor links to point to the correct place in the glossary, they must have an id
+  // This snippet ensures every definition term has an id
+  // See https://github.com/swcarpentry/styles/pull/129
   $('dt').each(function () {
     if (!this.id) {
       var id = $(this).text();
+      // If there's a ( in the name (e.g., "comma-separated values (CSV)") - just take everything up to the first (
       var index = id.indexOf('(');
       if (index > 0) {
         id = id.substring(0, index);
       }
+      // Strip leading and trailing whitespace, convert spaces to dashes and convert everything to lowercase
       id = id.trim().replace(/ /g, '-').toLowerCase();
       this.id = id;
     }
   });
-</script>
\ No newline at end of file
+</script>