From db7c28b20ecdfeedc1e6ce36a3f434dd4d72b019 Mon Sep 17 00:00:00 2001
From: squidfunk <martin.donath@squidfunk.com>
Date: Sun, 3 May 2020 14:14:59 +0200
Subject: [PATCH] Added name attribute to social links and improved defaults

---
 docs/getting-started.md       | 4 ++++
 material/partials/social.html | 9 ++++++---
 src/partials/social.html      | 9 ++++++---
 3 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/docs/getting-started.md b/docs/getting-started.md
index 838d470e5..1124fba5b 100644
--- a/docs/getting-started.md
+++ b/docs/getting-started.md
@@ -614,6 +614,10 @@ extra:
       link: https://linkedin.com/in/squidfunk
 ```
 
+By default, the link `title` will be set to the domain name, e.g. _github.com_.
+If you want to set a discernable name, e.g., to improve your Lighthouse score,
+you can set the `name` attribute on each social link.
+
   [21]: #icons
 
 ### Adding a Web App Manifest
diff --git a/material/partials/social.html b/material/partials/social.html
index 6c049e3af..80d650a8b 100644
--- a/material/partials/social.html
+++ b/material/partials/social.html
@@ -4,9 +4,12 @@
 {% if config.extra.social %}
   <div class="md-footer-social">
     {% for social in config.extra.social %}
-      {% set _,rest = social.link.split("//") %}
-      {% set domain = rest.split("/")[0] %}
-      <a href="{{ social.link }}" target="_blank" rel="noopener" title="{{ domain }}" class="md-footer-social__link">
+      {% set title = social.name %}
+      {% if not title and "//" in social.link %}
+        {% set _,url = social.link.split("//") %}
+        {% set title = url.split("/")[0] %}
+      {% endif %}
+      <a href="{{ social.link }}" target="_blank" rel="noopener" title="{{ title }}" class="md-footer-social__link">
         {% include ".icons/" ~ social.icon ~ ".svg" %}
       </a>
     {% endfor %}
diff --git a/src/partials/social.html b/src/partials/social.html
index 25ff9a766..abc10d019 100644
--- a/src/partials/social.html
+++ b/src/partials/social.html
@@ -24,12 +24,15 @@
 {% if config.extra.social %}
   <div class="md-footer-social">
     {% for social in config.extra.social %}
-      {% set _,rest = social.link.split("//") %}
-      {% set domain = rest.split("/")[0] %}
+      {% set title = social.name %}
+      {% if not title and "//" in social.link %}
+        {% set _,url = social.link.split("//") %}
+        {% set title = url.split("/")[0] %}
+      {% endif %}
       <a
         href="{{ social.link }}"
         target="_blank" rel="noopener"
-        title="{{ domain }}"
+        title="{{ title }}"
         class="md-footer-social__link"
       >
         {% include ".icons/" ~ social.icon ~ ".svg" %}