fix: handling of overlapping titles

lib/load-simple-icons.js

@@ -23,9 +23,19 @@ function loadSimpleIcons() {
       light: svg2base64(icon.svg.replace('<svg', `<svg fill="whitesmoke"`)),
       dark: svg2base64(icon.svg.replace('<svg', `<svg fill="#333"`)),
     }
+
+    // There are a few instances where multiple icons have the same title
+    // (e.g. 'Hive'). If a by-title reference we generate for
+    // backwards compatibility collides with a proper slug from Simple Icons
+    // then do nothing, so that the proper slug will always map to the correct icon.
+    if (!(title in originalSimpleIcons)) {
+      simpleIcons[title] = icon
+    }
+    if (!(legacyTitle in originalSimpleIcons)) {
+      simpleIcons[legacyTitle] = icon
+    }
+
     simpleIcons[key] = icon
-    simpleIcons[title] = icon
-    simpleIcons[legacyTitle] = icon
   })
   return simpleIcons
 }

lib/load-simple-icons.spec.js

@@ -30,4 +30,13 @@ describe('loadSimpleIcons', function () {
   it('excludes "get" function provided by the simple-icons', function () {
     expect(simpleIcons).to.not.have.property('get')
   })
+
+  it('maps overlapping icon titles correctly', function () {
+    // Both of these icons have the same title: 'Hive', so make sure
+    // the proper slugs are still mapped to the correct logo
+    expect(simpleIcons.hive.slug).to.equal('hive')
+    expect(simpleIcons.hive.title).to.equal('Hive')
+    expect(simpleIcons.hive_blockchain.slug).to.equal('hive_blockchain')
+    expect(simpleIcons.hive_blockchain.title).to.equal('Hive')
+  })
 })