cleaned up schema markup for books

1 files changed, 9 insertions, 4 deletions

diff --git a/design/templates/details/book.html b/design/templates/details/book.html
index 2ae6b86..b604df2 100644
--- a/design/templates/details/book.html
+++ b/design/templates/details/book.html
@@ -9,18 +9,23 @@
     <main itemprop="mainEntity" itemscope itemtype="http://schema.org/Book">
         <header class="tight">
         <h1 class="post-title book-title" itemprop="name">{{object.title|smartypants|widont|safe}}</h1>
-        <h2 class="post-subtitle">by {{object.author_name}}</h2>
+        <h2 class="post-subtitle" itemprop="author" itemscope itemtype="http://schema.org/Person">
+        <meta itemprop="name" content="{{object.author_name}}"/>by {{object.author_name}}</h2>
+
+
         </header>
         <div class="book-cover-wrapper">
             <img src="{{object.get_image_url}}" alt="{{object.title}} cover"  />
         </div>
         <div class="meta-cover">
             <dl class="book-metadata">
-                {% if object.rating %}<dt>Rating</dt><dd class="book-stars" itemprop="reviewRating"> {% for i in object.ratings_range %}{% if i <= object.get_rating%}&#9733;{%else%}&#9734;{%endif%}{%endfor%}</dd>{%endif%}
+
+                {% if object.rating %}<dt>Rating</dt><dd class="book-stars">
+          {% for i in object.ratings_range %}{% if i <= object.get_rating%}&#9733;{%else%}&#9734;{%endif%}{%endfor%}</span></dd>{%endif%}
                 {% if object.read_in %}<dt>Read</dt>
                 <dd>{{object.read_in}}</dd>{%endif%}
                 {% if object.pages %}<dt>Pages</dt>
-                <dd>{{object.pages}}</dd>{%endif%}
+                <dd itemprop="numberOfPages">{{object.pages}}</dd>{%endif%}
                 {% if object.publish_date %}<dt>Published</dt>
                 <dd>{%if object.publish_place%}{{object.publish_place}}, {%endif%}{{object.publish_date}}</dd>{%endif%}
                 {% if object.isbn %}<dt>ISBN</dt>
@@ -42,7 +47,7 @@
         <div class="highlights">
             <h4>Highlights:</h4>
             {% for object in object.bookhighlight_set.all reversed %}
-            <div class="h-entry hentry post--article book" itemscope itemType="http://schema.org/Article">
+            <div class="h-entry hentry post--article book">
                 <a class="highlink" id="h{{forloop.counter}}-page-{{object.page}}" href="#h{{forloop.counter}}-page-{{object.page}}">&para;</a>
                 <div class="book-highlight">
                     {{object.body_html|safe|amp|smartypants}}