From 58c07d16779d3d1f7a5f8c43518c41efad6e891b Mon Sep 17 00:00:00 2001
From: Joachim <joachim.robert@protonmail.com>
Date: Mon, 24 May 2021 12:36:48 +0200
Subject: [PATCH] Apply review suggestions

---
 bookwyrm/templates/author/author.html | 29 +++++++++++++--------------
 1 file changed, 14 insertions(+), 15 deletions(-)

diff --git a/bookwyrm/templates/author/author.html b/bookwyrm/templates/author/author.html
index 496012a82..de7d230f9 100644
--- a/bookwyrm/templates/author/author.html
+++ b/bookwyrm/templates/author/author.html
@@ -22,7 +22,7 @@
     </div>
 </div>
 
-<div class="block content columns" itemscope itemtype="https://schema.org/Person">
+<div class="block columns" itemscope itemtype="https://schema.org/Person">
     <meta itemprop="name" content="{{ author.name }}">
 
     {% if author.aliases or author.born or author.died or author.wikipedia_link or author.openlibrary_key or author.inventaire_id %}
@@ -32,11 +32,11 @@
                 {% if author.aliases %}
                 <div class="is-flex">
                     <dt class="mr-1">{% trans "Aliases:" %}</dt>
-                    <dd>
-                        {% for alias in author.aliases %}
-                        <span itemprop="alternateName">{{alias}}</span>{% if not forloop.last %},{% endif %}
-                        {% endfor %}
-                    </dd>
+                    {% for alias in author.aliases %}
+                        <dd itemprop="alternateName" content="{{alias}}">
+                            {{alias}}{% if not forloop.last %},{% endif %}
+                        </dd>
+                    {% endfor %}
                 </div>
                 {% endif %}
                 {% if author.born %}
@@ -54,25 +54,24 @@
             </dl>
 
             {% if author.wikipedia_link %}
-            <p>
-                <link itemprop="sameAs" href="{{ author.wikipedia_link }}" />
-                <a href="{{ author.wikipedia_link }}" rel=”noopener” target="_blank">
+            <p class="mt-1">
+                <a itemprop="sameAs" href="{{ author.wikipedia_link }}" rel=”noopener” target="_blank">
                     {% trans "Wikipedia" %}
                 </a>
             </p>
             {% endif %}
+            
             {% if author.openlibrary_key %}
-            <p class="mb-0">
-                <link itemprop="sameAs" href="https://openlibrary.org/authors/{{ author.openlibrary_key }}" />
-                <a href="https://openlibrary.org/authors/{{ author.openlibrary_key }}" target="_blank" rel="noopener">
+            <p class="mt-1">
+                <a itemprop="sameAs" href="https://openlibrary.org/authors/{{ author.openlibrary_key }}" target="_blank" rel="noopener">
                     {% trans "View on OpenLibrary" %}
                 </a>
             </p>
             {% endif %}
+            
             {% if author.inventaire_id %}
-            <p class="mb-0">
-                <link itemprop="sameAs" href="https://inventaire.io/entity/{{ author.inventaire_id }}" />
-                <a href="https://inventaire.io/entity/{{ author.inventaire_id }}" target="_blank" rel="noopener">
+            <p class="mt-1">
+                <a itemprop="sameAs" href="https://inventaire.io/entity/{{ author.inventaire_id }}" target="_blank" rel="noopener">
                     {% trans "View on Inventaire" %}
                 </a>
             </p>