Merge pull request #292 from mouse-reeve/markup-cleanup

Markup cleanup

bookwyrm/static/css/fonts/icomoon.svg

@@ -30,6 +30,7 @@
 <glyph unicode="&#xe914;" glyph-name="home" horiz-adv-x="951" d="M804.571 384v-274.286c0-20-16.571-36.571-36.571-36.571h-219.429v219.429h-146.286v-219.429h-219.429c-20 0-36.571 16.571-36.571 36.571v274.286c0 1.143 0.571 2.286 0.571 3.429l328.571 270.857 328.571-270.857c0.571-1.143 0.571-2.286 0.571-3.429zM932 423.428l-35.429-42.286c-2.857-3.429-7.429-5.714-12-6.286h-1.714c-4.571 0-8.571 1.143-12 4l-395.429 329.714-395.429-329.714c-4-2.857-8.571-4.571-13.714-4-4.571 0.571-9.143 2.857-12 6.286l-35.429 42.286c-6.286 7.429-5.143 19.429 2.286 25.714l410.857 342.286c24 20 62.857 20 86.857 0l139.429-116.571v111.429c0 10.286 8 18.286 18.286 18.286h109.714c10.286 0 18.286-8 18.286-18.286v-233.143l125.143-104c7.429-6.286 8.571-18.286 2.286-25.714z" />
 <glyph unicode="&#xe915;" glyph-name="local" horiz-adv-x="1097" d="M338.857 438.857c-59.429-1.714-113.143-27.429-151.429-73.143h-76.571c-57.143 0-110.857 27.429-110.857 90.857 0 46.286-1.714 201.714 70.857 201.714 12 0 71.429-48.571 148.571-48.571 26.286 0 51.429 4.571 76 13.143-1.714-12.571-2.857-25.143-2.857-37.714 0-52 16.571-103.429 46.286-146.286zM950.857 74.857c0-92.571-61.143-148-152.571-148h-499.429c-91.429 0-152.571 55.429-152.571 148 0 129.143 30.286 327.429 197.714 327.429 19.429 0 90.286-79.429 204.571-79.429s185.143 79.429 204.571 79.429c167.429 0 197.714-198.286 197.714-327.429zM365.714 804.571c0-80.571-65.714-146.286-146.286-146.286s-146.286 65.714-146.286 146.286 65.714 146.286 146.286 146.286 146.286-65.714 146.286-146.286zM768 585.143c0-121.143-98.286-219.429-219.429-219.429s-219.429 98.286-219.429 219.429 98.286 219.429 219.429 219.429 219.429-98.286 219.429-219.429zM1097.143 456.571c0-63.429-53.714-90.857-110.857-90.857h-76.571c-38.286 45.714-92 71.429-151.429 73.143 29.714 42.857 46.286 94.286 46.286 146.286 0 12.571-1.143 25.143-2.857 37.714 24.571-8.571 49.714-13.143 76-13.143 77.143 0 136.571 48.571 148.571 48.571 72.571 0 70.857-155.429 70.857-201.714zM1024 804.571c0-80.571-65.714-146.286-146.286-146.286s-146.286 65.714-146.286 146.286 65.714 146.286 146.286 146.286 146.286-65.714 146.286-146.286z" />
 <glyph unicode="&#xe916;" glyph-name="dots-three" d="M512.051 573.44c-62.208 0-112.691-50.432-112.691-112.64s50.483-112.64 112.691-112.64c62.208 0 112.589 50.432 112.589 112.64s-50.381 112.64-112.589 112.64zM153.651 573.44c-62.208 0-112.691-50.432-112.691-112.64s50.483-112.64 112.691-112.64c62.208 0 112.589 50.483 112.589 112.64s-50.381 112.64-112.589 112.64zM870.451 573.44c-62.208 0-112.691-50.432-112.691-112.64s50.483-112.64 112.691-112.64c62.208 0 112.589 50.432 112.589 112.64s-50.381 112.64-112.589 112.64z" />
+<glyph unicode="&#xe917;" glyph-name="check" d="M424.653 102.502c-22.272 0-43.366 10.394-56.883 28.314l-182.938 241.715c-23.808 31.386-17.613 76.083 13.824 99.891 31.488 23.91 76.186 17.613 99.994-13.824l120.371-158.925 302.643 485.99c20.838 33.382 64.87 43.622 98.355 22.784 33.434-20.787 43.725-64.819 22.835-98.304l-357.581-573.952c-12.39-20.019-33.843-32.512-57.344-33.587-1.126-0.102-2.15-0.102-3.277-0.102z" />
 <glyph unicode="&#xe986;" glyph-name="search" d="M992.262 88.604l-242.552 206.294c-25.074 22.566-51.89 32.926-73.552 31.926 57.256 67.068 91.842 154.078 91.842 249.176 0 212.078-171.922 384-384 384-212.076 0-384-171.922-384-384s171.922-384 384-384c95.098 0 182.108 34.586 249.176 91.844-1-21.662 9.36-48.478 31.926-73.552l206.294-242.552c35.322-39.246 93.022-42.554 128.22-7.356s31.892 92.898-7.354 128.22zM384 320c-141.384 0-256 114.616-256 256s114.616 256 256 256 256-114.616 256-256-114.614-256-256-256z" />
 <glyph unicode="&#xe9d7;" glyph-name="star-empty" d="M1024 562.95l-353.78 51.408-158.22 320.582-158.216-320.582-353.784-51.408 256-249.538-60.432-352.352 316.432 166.358 316.432-166.358-60.434 352.352 256.002 249.538zM512 206.502l-223.462-117.48 42.676 248.83-180.786 176.222 249.84 36.304 111.732 226.396 111.736-226.396 249.836-36.304-180.788-176.222 42.678-248.83-223.462 117.48z" />
 <glyph unicode="&#xe9d8;" glyph-name="star-half" d="M1024 562.95l-353.78 51.408-158.22 320.582-158.216-320.582-353.784-51.408 256-249.538-60.432-352.352 316.432 166.358 316.432-166.358-60.434 352.352 256.002 249.538zM512 206.502l-0.942-0.496 0.942 570.768 111.736-226.396 249.836-36.304-180.788-176.222 42.678-248.83-223.462 117.48z" />

bookwyrm/static/css/icons.css

@@ -1,10 +1,10 @@
 @font-face {
   font-family: 'icomoon';
-  src:  url('fonts/icomoon.eot?ouoizu');
-  src:  url('fonts/icomoon.eot?ouoizu#iefix') format('embedded-opentype'),
-    url('fonts/icomoon.ttf?ouoizu') format('truetype'),
-    url('fonts/icomoon.woff?ouoizu') format('woff'),
-    url('fonts/icomoon.svg?ouoizu#icomoon') format('svg');
+  src:  url('fonts/icomoon.eot?aeb2zb');
+  src:  url('fonts/icomoon.eot?aeb2zb#iefix') format('embedded-opentype'),
+    url('fonts/icomoon.ttf?aeb2zb') format('truetype'),
+    url('fonts/icomoon.woff?aeb2zb') format('woff'),
+    url('fonts/icomoon.svg?aeb2zb#icomoon') format('svg');
   font-weight: normal;
   font-style: normal;
   font-display: block;
@@ -25,6 +25,9 @@
   -moz-osx-font-smoothing: grayscale;
 }
 
+.icon-check:before {
+  content: "\e917";
+}
 .icon-dots-three:before {
   content: "\e916";
 }

bookwyrm/templates/book.html

@@ -27,11 +27,18 @@
             {% include 'snippets/shelve_button.html' %}
 
             {% if request.user.is_authenticated and not book.cover %}
-            <form name="add-cover" method="POST" action="/upload_cover/{{ book.id }}" enctype="multipart/form-data">
-                {% csrf_token %}
-                {{ cover_form.as_p }}
-                <button class="button" type="submit">Add cover</button>
-            </form>
+            <div class="box p-2">
+                <form name="add-cover" method="POST" action="/upload_cover/{{ book.id }}" enctype="multipart/form-data">
+                    {% csrf_token %}
+                    <div class="field">
+                        <label class="label" for="id_cover">Cover:</label>
+                        <input type="file" name="cover" accept="image/*" class="" id="id_cover">
+                    </div>
+                    <div class="field">
+                        <button class="button is-small" type="submit">Add cover</button>
+                    </div>
+                </form>
+            </div>
             {% endif %}
 
             <dl class="content">
@@ -60,59 +67,85 @@
                 <input class="toggle-control" type="radio" name="show-edit-readthrough" id="show-readthrough-{{ readthrough.id }}" checked>
                 <div class="toggle-content hidden">
                     <dl>
+                        {% if readthrough.start_date %}
                         <dt>Started reading:</dt>
                         <dd>{{ readthrough.start_date | naturalday }}</dd>
+                        {% endif %}
+                        {% if readthrough.finish_date %}
                         <dt>Finished reading:</dt>
                         <dd>{{ readthrough.finish_date | naturalday }}</dd>
+                        {% endif %}
                     </dl>
                     <div class="field is-grouped">
                         <label class="button is-small" for="edit-readthrough-{{ readthrough.id }}">
                             <span class="icon icon-pencil">
-                                <span class="is-sr-only">Edit readthrough dates</span>
+                                <span class="is-sr-only">Edit read-through dates</span>
                             </span>
                         </label>
-                        <form name="delete-readthrough-{{ readthrough.id }}" action="/delete-readthrough" method="POST">
-                            {% csrf_token %}
-                            <input type="hidden" name="id" value="{{ readthrough.id }}">
-                            <button class="button is-small" type="submit">
-                                <span class="icon icon-x">
-                                    <span class="is-sr-only">Delete this readthrough</span>
-                                </span>
-                            </button>
+                        <label class="button is-small" for="delete-readthrough-{{ readthrough.id }}">
+                            <span class="icon icon-x">
+                                <span class="is-sr-only">Delete this read-through</span>
+                            </span>
+                        </label>
+                    </div>
+                </div>
+            </div>
+
+            <div class="block">
+                <input class="toggle-control" type="radio" name="show-edit-readthrough" id="edit-readthrough-{{ readthrough.id }}">
+                <div class="toggle-content hidden">
+                    <div class="box">
+                        <form name="edit-readthrough" action="/edit-readthrough" method="post">
+                        {% csrf_token %}
+                        <input type="hidden" name="id" value="{{ readthrough.id }}">
+                            <div class="field">
+                                <label class="label" for="start_date">
+                                    Started reading
+                                    <input type="date" name="start_date" class="input" id="id_start_date-{{ readthrough.id }}" value="{{ readthrough.start_date | date:"Y-m-d" }}">
+                                </label>
+                            </div>
+                            <div class="field">
+                                <label class="label" for="finish_date">
+                                    Finished reading
+                                    <input type="date" name="finish_date" class="input" id="id_finish_date-{{ readthrough.id }}" value="{{ readthrough.finish_date | date:"Y-m-d" }}">
+                                </label>
+                            </div>
+                            <div class="field is-grouped">
+                                <button class="button is-primary" type="submit">Save</button>
+                                <label class="button" for="show-readthrough-{{ readthrough.id }}">Cancel</label>
+                            </div>
                         </form>
                     </div>
                 </div>
             </div>
 
-            <div class="content block">
-                <input class="toggle-control" type="radio" name="show-edit-readthrough" id="edit-readthrough-{{ readthrough.id }}">
-                <div class="toggle-content hidden">
-                    <form name="edit-readthrough" action="/edit-readthrough" method="post">
-                        {% csrf_token %}
-                        <input type="hidden" name="id" value="{{ readthrough.id }}">
-                        <div class="field">
-                            <label class="label" for="start_date">
-                                Started reading
-                                <input type="date" name="start_date" class="input" id="id_start_date-{{ readthrough.id }}" value="{{ readthrough.start_date | date:"Y-m-d" }}">
-                            </label>
-                        </div>
-                        <div class="field">
-                            <label class="label" for="finish_date">
-                                Finished reading
-                                <input type="date" name="finish_date" class="input" id="id_finish_date-{{ readthrough.id }}" value="{{ readthrough.finish_date | date:"Y-m-d" }}">
-                            </label>
-                        </div>
-                        <div class="field is-grouped">
-                            <button class="button is-small" type="submit">Save</button>
-                            <label class="button is-small" for="show-readthrough-{{ readthrough.id }}">Cancel</label>
-                        </div>
-                    </form>
+            <div>
+                <input class="toggle-control" type="checkbox" name="delete-readthrough-{{ readthrough.id }}" id="delete-readthrough-{{ readthrough.id }}">
+                <div class="modal toggle-content hidden">
+                    <div class="modal-background"></div>
+                    <div class="modal-card">
+                        <header class="modal-card-head">
+                            <p class="modal-card-title">Delete this read-though?</p>
+                            <label class="delete" for="delete-readthrough-{{ readthrough.id }}" aria-label="close"></label>
+                        </header>
+                        <footer class="modal-card-foot">
+			    <form name="delete-readthrough-{{ readthrough.id }}" action="/delete-readthrough" method="POST">
+				{% csrf_token %}
+				<input type="hidden" name="id" value="{{ readthrough.id }}">
+				<button class="button is-danger" type="submit">
+				    Delete
+				</button>
+				<label for="delete-readthrough-{{ readthrough.id }}" class="button">Cancel</button>
+			    </form>
+                        </footer>
+                    </div>
+                    <label class="modal-close is-large" for="delete-readthrough-{{ readthrough.id }}" aria-label="close"></label>
                 </div>
             </div>
             {% endfor %}
 
             {% if request.user.is_authenticated %}
-            <div class="block">
+            <div class="box">
                 {% include 'snippets/create_status.html' with book=book hide_cover=True %}
             </div>
 

bookwyrm/templates/followers.html

@@ -1,7 +1,7 @@
 {% extends 'layout.html' %}
 {% load fr_display %}
 {% block content %}
-{% include 'user_header.html' with user=user %}
+{% include 'snippets/user_header.html' with user=user %}
 
 <div class="block">
     <h2 class="title">Followers</h2>

bookwyrm/templates/following.html

@@ -1,7 +1,7 @@
 {% extends 'layout.html' %}
 {% load fr_display %}
 {% block content %}
-{% include 'user_header.html' %}
+{% include 'snippets/user_header.html' with user=user %}
 
 <div class="block">
     <h2 class="title">Following</h2>

bookwyrm/templates/login.html

@@ -57,6 +57,5 @@
             </p>
         </div>
     </div>
-
 </div>
 {% endblock %}

bookwyrm/templates/manage_invites.html

@@ -29,21 +29,18 @@
 
     <form name="invite" action="/create_invite/" method="post">
         {% csrf_token %}
-        <div class="field">
+        <div class="field is-grouped">
             <div class="control">
                 <label class="label" for="id_expiry">Expiry:</label>
+                <div class="select">
+                    {{ form.expiry }}
+                </div>
             </div>
-            <div class="select">
-                {{ form.expiry }}
-            </div>
-        </div>
-
-        <div class="field">
             <div class="control">
                 <label class="label" for="id_use_limit">Use limit:</label>
-            </div>
-            <div class="select">
-                {{ form.use_limit }}
+                <div class="select">
+                    {{ form.use_limit }}
+                </div>
             </div>
         </div>
 

bookwyrm/templates/notifications.html

@@ -59,4 +59,3 @@
 </div>
 
 {% endblock %}
-

bookwyrm/templates/password_reset_request.html

@@ -17,7 +17,7 @@
                 </div>
                 <div class="field is-grouped">
                     <div class="control">
-                        <button class="button is-primary" type="submit">Reset password</button>
+                        <button class="button is-link" type="submit">Reset password</button>
                     </div>
                 </div>
             </form>

bookwyrm/templates/search_results.html

@@ -49,7 +49,7 @@
                             {% csrf_token %}
                             <input type="hidden" name="remote_id" value="{{ result.key }}">
                             <div>{% include 'snippets/search_result_text.html' with result=result link=False %}</div>
-                            <button type="submit" class="button is-small is-link is-light">Import book</button>
+                            <button type="submit" class="button is-small is-link">Import book</button>
                         </form>
                     </li>
                 {% endfor %}

bookwyrm/templates/shelf.html

@@ -1,7 +1,7 @@
 {% extends 'layout.html' %}
 {% load fr_display %}
 {% block content %}
-{% include 'user_header.html' with user=user %}
+{% include 'snippets/user_header.html' with user=user %}
 
 <div class="block">
     <div class="tabs">

bookwyrm/templates/snippets/create_status.html

@@ -1,109 +1,31 @@
 {% load humanize %}
 {% load fr_display %}
 
-<div class="columns">
-    <div class="column">
-        <div class="tabs is-boxed">
-            <ul>
-                <li class="is-active" data-id="tab-review-{{ book.id }}" data-category="tab-option-{{ book.id }}">
-                    <label for="review-{{ book.id }}" onclick="tabChange(event)"><a>Review</a></label>
-                </li>
-                <li data-id="tab-comment-{{ book.id }}" data-category="tab-option-{{ book.id }}">
-                    <label for="comment-{{ book.id}}" onclick="tabChange(event)"><a>Comment</a></label>
-                </li>
-                <li data-id="tab-quotation-{{ book.id }}" data-category="tab-option-{{ book.id }}">
-                    <label for="quote-{{ book.id }}" onclick="tabChange(event)"><a>Quote</a></label>
-                </li>
-            </ul>
-        </div>
-
-        <div>
-            <input class="toggle-control" type="radio" name="status-tabs-{{ book.id }}" id="review-{{ book.id }}" checked>
-            <form class="toggle-content hidden tab-option-{{ book.id }}" name="review" action="/review/" method="post" id="tab-review-{{ book.id }}">
-                {% csrf_token %}
-                <input type="hidden" name="book" value="{{ book.id }}">
-                <input type="hidden" name="user" value="{{ request.user.id }}">
-                <div class="control">
-                    <label class="label" for="id_name_{{ book.id }}_review">Title:</label>
-                    <input type="text" name="name" maxlength="255" class="input" required="" id="id_name_{{ book.id }}_review" placeholder="My review of '{{ book.title }}'">
-                </div>
-                <div class="control">
-                    <label class="label" for="id_content_{{ book.id }}_review">Review:</label>
-
-                    <span class="is-sr-only">Rating</span>
-                    <div class="field is-grouped stars form-rate-stars">
-                        <input class="hidden" type="radio" name="rating" value="" checked>
-                        {% for i in '12345'|make_list %}
-                            <input class="hidden" id="book{{book.id}}-star-{{ forloop.counter }}" type="radio" name="rating" value="{{ forloop.counter }}">
-                            <label class="icon icon-star-empty" for="book{{book.id}}-star-{{ forloop.counter }}">
-                                <span class="is-sr-only">{{ forloop.counter }} star{{ forloop.counter | pluralize }}</span>
-                            </label>
-                        {% endfor %}
-                    </div>
-
-                    <textarea name="content" class="textarea" id="id_content_{{ book.id }}_review"></textarea>
-                </div>
-                <div class="control is-grouped">
-                    {% include 'snippets/privacy_select.html' %}
-                    <button class="button is-primary" type="submit">post review</button>
-                </div>
-            </form>
-        </div>
-
-        <div>
-            <input class="toggle-control" type="radio" name="status-tabs-{{ book.id }}" id="comment-{{ book.id }}">
-            <form class="toggle-content hidden tab-option-{{ book.id }}" name="comment" action="/comment/" method="post" id="tab-comment-{{ book.id }}">
-                {% csrf_token %}
-                <input type="hidden" name="book" value="{{ book.id }}">
-                <input type="hidden" name="book" value="{{ book.id }}">
-                <input type="hidden" name="user" value="{{ request.user.id }}">
-                <input type="hidden" name="privacy" value="public">
-                <div class="control">
-                    <label class="label" for="id_content_{{ book.id }}_comment">Comment:</label>
-                    <textarea name="content" class="textarea" id="id_content_{{ book.id }}_comment" placeholder="Some thoughts on '{{ book.title }}'"></textarea>
-                </div>
-                <div class="control is-grouped">
-                    <div class="select">
-                        <select name="privacy">
-                            <option value="public" selected>Public</option>
-                            <option value="unlisted">Unlisted</option>
-                            <option value="followers">Followers only</option>
-                            <option value="direct">Private</option>
-                        </select>
-                    </div>
-                    <button class="button is-primary" type="submit">post comment</button>
-                </div>
-            </form>
-        </div>
-
-        <div>
-            <input class="toggle-control" type="radio" name="status-tabs-{{ book.id }}" id="quote-{{ book.id }}">
-            <form class="toggle-content hidden tab-option-{{ book.id }}" name="quotation" action="/quotate/" method="post" id="tab-quotation-{{ book.id }}">
-                {% csrf_token %}
-                <input type="hidden" name="book" value="{{ book.id }}">
-                <input type="hidden" name="book" value="{{ book.id }}">
-                <input type="hidden" name="user" value="{{ request.user.id }}">
-                <input type="hidden" name="privacy" value="public">
-                <div class="control">
-                    <label class="label" for="id_quote_{{ book.id }}_quote">Quote:</label>
-                    <textarea name="quote" class="textarea" required="" id="id_quote_{{ book.id }}_quote" placeholder="An except from '{{ book.title }}'"></textarea>
-                </div>
-                <div class="control">
-                    <label class="label" for="id_content_{{ book.id }}_quote">Comment:</label>
-                    <textarea name="content" class="textarea is-small" id="id_content_{{ book.id }}_quote"></textarea>
-                </div>
-                <div class="control is-grouped">
-                    <div class="select">
-                        <select name="privacy">
-                            <option value="public" selected>Public</option>
-                            <option value="unlisted">Unlisted</option>
-                            <option value="followers">Followers only</option>
-                            <option value="direct">Private</option>
-                        </select>
-                    </div>
-                    <button class="button is-primary" type="submit">post quote</button>
-                </div>
-            </form>
-        </div>
-    </div>
+<div class="tabs is-boxed">
+    <ul>
+        <li class="is-active" data-id="tab-review-{{ book.id }}" data-category="tab-option-{{ book.id }}">
+            <label for="review-{{ book.id }}" onclick="tabChange(event)"><a>Review</a></label>
+        </li>
+        <li data-id="tab-comment-{{ book.id }}" data-category="tab-option-{{ book.id }}">
+            <label for="comment-{{ book.id}}" onclick="tabChange(event)"><a>Comment</a></label>
+        </li>
+        <li data-id="tab-quotation-{{ book.id }}" data-category="tab-option-{{ book.id }}">
+            <label for="quote-{{ book.id }}" onclick="tabChange(event)"><a>Quote</a></label>
+        </li>
+    </ul>
+</div>
+
+<div>
+    <input class="toggle-control" type="radio" name="status-tabs-{{ book.id }}" id="review-{{ book.id }}" checked>
+    {% include 'snippets/create_status_form.html' with type='review' %}
+</div>
+
+<div>
+    <input class="toggle-control" type="radio" name="status-tabs-{{ book.id }}" id="comment-{{ book.id }}">
+    {% include 'snippets/create_status_form.html' with type="comment" placeholder="Some thougts on '"|add:book.title|add:"'" %}
+</div>
+
+<div>
+    <input class="toggle-control" type="radio" name="status-tabs-{{ book.id }}" id="quote-{{ book.id }}">
+    {% include 'snippets/create_status_form.html' with type="quote" placeholder="An excerpt from '"|add:book.title|add:"'" %}
 </div>

bookwyrm/templates/snippets/create_status_form.html

@@ -0,0 +1,40 @@
+<form class="toggle-content hidden tab-option-{{ book.id }}" name="{{ type }}" action="/{{ type }}" method="post" id="tab-{{ type }}-{{ book.id }}">
+    {% csrf_token %}
+    <input type="hidden" name="book" value="{{ book.id }}">
+    <input type="hidden" name="user" value="{{ request.user.id }}">
+    {% if type == 'review' %}
+    <div class="control">
+        <label class="label" for="id_name_{{ book.id }}_{{ type }}">Title:</label>
+        <input type="text" name="name" maxlength="255" class="input" required="" id="id_name_{{ book.id }}_{{ type }}" placeholder="My {{ type }} of '{{ book.title }}'">
+    </div>
+    {% endif %}
+    <div class="control">
+        <label class="label" for="id_content_{{ book.id }}_{{ type }}">{{ type|title }}:</label>
+
+        {% if type == 'review' %}
+        <span class="is-sr-only">Rating</span>
+        <div class="field is-grouped stars form-rate-stars">
+            <input class="hidden" type="radio" name="rating" value="" checked>
+            {% for i in '12345'|make_list %}
+                <input class="hidden" id="book{{book.id}}-star-{{ forloop.counter }}" type="radio" name="rating" value="{{ forloop.counter }}">
+                <label class="icon icon-star-empty" for="book{{book.id}}-star-{{ forloop.counter }}">
+                    <span class="is-sr-only">{{ forloop.counter }} star{{ forloop.counter | pluralize }}</span>
+                </label>
+            {% endfor %}
+        </div>
+        {% endif %}
+        <textarea name="{% if type == 'quote' %}quote{% else %}content{% endif %}" class="textarea" id="id_content_{{ book.id }}_{{ type }}" placeholder="{{ placeholder }}" required></textarea>
+
+    </div>
+    {% if type == 'quote' %}
+    <div class="control">
+        <label class="label" for="id_content_{{ book.id }}_quote">Comment:</label>
+        <textarea name="content" class="textarea is-small" id="id_content_{{ book.id }}_quote"></textarea>
+    </div>
+    {% endif %}
+    <div class="control is-grouped">
+        {% include 'snippets/privacy_select.html' %}
+        <button class="button is-primary" type="submit">post {{ type }}</button>
+    </div>
+</form>
+

bookwyrm/templates/snippets/follow_button.html

@@ -11,14 +11,14 @@ Follow request already sent.
     {% csrf_token %}
     <input type="hidden" name="user" value="{{ user.username }}">
     {% if user.manually_approves_followers %}
-        <button class="button is-small is-link is-light" type="submit">Send follow request</button>
+        <button class="button is-small is-primary" type="submit">Send follow request</button>
     {% else %}
-        <button class="button is-small is-link is-light" type="submit">Follow</button>
+        <button class="button is-small is-primary" type="submit">Follow</button>
     {% endif %}
 </form>
 <form action="/unfollow/" method="POST" onsubmit="interact(event)" class="follow-{{ user.id }} {% if not request.user in user.followers.all %}hidden{%endif %}" data-id="follow-{{ user.id }}">
     {% csrf_token %}
     <input type="hidden" name="user" value="{{ user.username }}">
-    <button class="button is-small is-danger is-light" type="submit">Unfollow</button>
+    <button class="button is-small is-danger" type="submit">Unfollow</button>
 </form>
 {% endif %}

bookwyrm/templates/snippets/follow_request_buttons.html

@@ -3,11 +3,11 @@
 <form action="/accept_follow_request/" method="POST">
     {% csrf_token %}
     <input type="hidden" name="user" value="{{ user.username }}">
-    <button class="button is-small" type="submit">Accept</button>
+    <button class="button is-primary is-small" type="submit">Accept</button>
 </form>
 <form action="/delete_follow_request/" method="POST">
     {% csrf_token %}
     <input type="hidden" name="user" value="{{ user.username }}">
-    <button class="button is-small" type="submit" class="warning">Delete</button>
+    <button class="button is-danger is-small" type="submit" class="warning">Delete</button>
 </form>
 {% endif %}

bookwyrm/templates/snippets/shelf.html

@@ -71,7 +71,7 @@
             {% csrf_token %}
             <input type="hidden" name="book" value="{{ book.id }}">
             <input type="hidden" name="shelf" value="{{ shelf.id }}">
-            <button class="button is-small" type="submit" style="">
+            <button class="button is-danger is-small" type="submit" style="">
                 <span class="icon icon-x">
                     <span class="is-sr-only">Remove from shelf</span>
                 </span>

bookwyrm/templates/snippets/shelve_button.html

@@ -5,7 +5,9 @@
 {% active_shelf book as active_shelf %}
 <div class="field is-grouped">
     {% if active_shelf.identifier == 'read' %}
-    <button class="button is-small" disabled>Read ✓</button>
+    <button class="button is-small" disabled>
+        <span>Read</span> <span class="icon icon-check"></span>
+    </button>
     {% elif active_shelf.identifier == 'reading' %}
     <label class="button is-small" for="finish-reading-{{ uuid }}">
         I'm done!
@@ -42,7 +44,10 @@
                     <form class="dropdown-item pt-0 pb-0" name="shelve" action="/shelve/" method="post">
                         {% csrf_token %}
                         <input type="hidden" name="book" value="{{ book.id }}">
-                        <button class="button is-small" name="shelf" type="submit" value="{{ shelf.identifier }}" {% if shelf in book.shelf_set.all %} disabled {% endif %}>{{ shelf.name }} {% if shelf in book.shelf_set.all %} ✓ {% endif %}</button>
+                        <button class="button is-small" name="shelf" type="submit" value="{{ shelf.identifier }}" {% if shelf in book.shelf_set.all %} disabled {% endif %}>
+                            <span>{{ shelf.name }}</span>
+                            {% if shelf in book.shelf_set.all %}<span class="icon icon-check"></span>{% endif %}
+                        </button>
                     </form>
                     {% endif %}
                 </li>
@@ -58,7 +63,7 @@
         <div class="modal-background"></div>
         <div class="modal-card">
             <header class="modal-card-head">
-                <p class="modal-card-title">Started "{{ book.title }}"</p>
+                <p class="modal-card-title">Start "{{ book.title }}"</p>
                 <label class="delete" for="start-reading-{{ uuid }}" aria-label="close"></label>
             </header>
             <form name="start-reading" action="/start-reading" method="post">
@@ -82,14 +87,14 @@
                             {% include 'snippets/privacy_select.html' %}
                         </div>
                         <div class="column">
-                            <button type="submit" class="button is-success">Save</button>
+                            <button class="button is-success" type="submit">Save</button>
                             <label for="start-reading-{{ uuid }}" class="button">Cancel</button>
                         </div>
                     </div>
                 </footer>
             </form>
         </div>
-        <label class="modal-close is-large" for="finish-reading-{{ uuid }}" aria-label="close"></label>
+        <label class="modal-close is-large" for="start-reading-{{ uuid }}" aria-label="close"></label>
     </div>
 </div>
 
@@ -99,7 +104,7 @@
         <div class="modal-background"></div>
         <div class="modal-card">
             <header class="modal-card-head">
-                <p class="modal-card-title">Finished "{{ book.title }}"</p>
+                <p class="modal-card-title">Finish "{{ book.title }}"</p>
                 <label class="delete" for="finish-reading-{{ uuid }}" aria-label="close"></label>
             </header>
             {% active_read_through book user as readthrough %}

bookwyrm/templates/snippets/status.html

@@ -5,18 +5,20 @@
 <div class="card">
     <header class="card-header">
         <div class="card-header-title">
-            <p>
-                {% if status.status_type == 'Boost' %}
-                {% include 'snippets/avatar.html' with user=status.user %}
-                {% include 'snippets/username.html' with user=status.user %}
-                boosted
-            </p>
-            <p>
-                {% include 'snippets/status_header.html' with status=status|boosted_status %}
-                {% else %}
-                {% include 'snippets/status_header.html' with status=status %}
-                {% endif %}
-            </p>
+            <div class="columns">
+                <div class="column is-narrow">
+                    {% if status.status_type == 'Boost' %}
+                    {% include 'snippets/avatar.html' with user=status.user %}
+                    {% include 'snippets/username.html' with user=status.user %}
+                    boosted
+                </div>
+                <div class="column">
+                    {% include 'snippets/status_header.html' with status=status|boosted_status %}
+                    {% else %}
+                    {% include 'snippets/status_header.html' with status=status %}
+                    {% endif %}
+                </div>
+            </div>
         </div>
     </header>
 
@@ -117,7 +119,7 @@
                     <form name="delete-{{status.id}}" action="/delete-status" method="post">
                         {% csrf_token %}
                         <input type="hidden" name="status" value="{{ status.id }}">
-                        <button type="submit" class="button">
+                        <button class="button is-danger" type="submit">
                             Delete post
                         </button>
                     </form>

bookwyrm/templates/snippets/tag.html

@@ -1,24 +1,22 @@
 <div class="control">
-    <div class="tags has-addons">
-        <a class="tag is-link" href="/tag/{{ tag.identifier|urlencode }}">
-            {{ tag.name }}
-        </a>
-        <div class="tag">
+    <form name="tag" action="/{% if tag.identifier in user_tags %}untag{% else %}tag{% endif %}/" method="post">
+        {% csrf_token %}
+        <input type="hidden" name="book" value="{{ book.id }}">
+        <input type="hidden" name="name" value="{{ tag.name }}">
+
+        <div class="tags has-addons">
+            <a class="tag" href="/tag/{{ tag.identifier|urlencode }}">
+                {{ tag.name }}
+            </a>
             {% if tag.identifier in user_tags %}
-            <form name="tag" action="/untag/" method="post">
-                {% csrf_token %}
-                <input type="hidden" name="book" value="{{ book.id }}">
-                <input type="hidden" name="name" value="{{ tag.name }}">
-                <button type="submit">x<span class="is-sr-only"> remove tag</span></button>
-            </form>
+            <button class="tag is-delete" type="submit">
+                <span class="is-sr-only">remove tag</span>
+            </button>
             {% else %}
-            <form name="tag" action="/tag/" method="post">
-                {% csrf_token %}
-                <input type="hidden" name="book" value="{{ book.id }}">
-                <input type="hidden" name="name" value="{{ tag.name }}">
-                <button type="submit">+<span class="is-sr-only"> add tag</span></button>
-            </form>
+            <button class="tag" type="submit">+
+                <span class="is-sr-only">add tag</span>
+            </button>
             {% endif %}
         </div>
-    </div>
+    </form>
 </div>

bookwyrm/templates/snippets/user_header.html

@@ -18,15 +18,18 @@
         <div class="column is-narrow">
             <div class="media">
                 <div class="media-left">
+                    <a href="/user/{{ user|username }}">
                     {% include 'snippets/avatar.html' with user=user large=True %}
+                    </a>
                 </div>
                 <div class="media-content">
                     <p>{% if user.name %}{{ user.name }}{% else %}{{ user.localname }}{% endif %}</p>
-                    <p>{{ user.username }}</p>
+                    <p><a href="{{ user.remote_id }}">{{ user.username }}</a></p>
                     <p>Joined {{ user.created_date | naturaltime }}</p>
                     <p>
-                    <a href="/user/{{ user | username }}/followers">{{ user.followers.count }} follower{{ user.followers.count | pluralize }}</a>,
-                        <a href="/user/{{ user | username }}/following">{{ user.following.count }} following</a></p>
+                        <a href="/user/{{ user | username }}/followers">{{ user.followers.count }} follower{{ user.followers.count | pluralize }}</a>,
+                        <a href="/user/{{ user | username }}/following">{{ user.following.count }} following</a>
+                    </p>
                 </div>
             </div>
         </div>

bookwyrm/templates/user.html

@@ -1,9 +1,7 @@
 {% extends 'layout.html' %}
 {% block content %}
 
-<div class="block">
-    {% include 'user_header.html' with user=user %}
-</div>
+{% include 'snippets/user_header.html' with user=user %}
 
 <div class="block">
     <h2 class="title">Shelves</h2>

bookwyrm/templates/user_shelves.html

@@ -1,7 +1,7 @@
 {% extends 'layout.html' %}
 {% load fr_display %}
 {% block content %}
-{% include 'user_header.html' with user=user %}
+{% include 'snippets/user_header.html' with user=user %}
 
 <div class="block">
     <div class="tabs">

bookwyrm/urls.py

@@ -107,7 +107,7 @@ urlpatterns = [
 
     re_path(r'^rate/?$', actions.rate),
     re_path(r'^review/?$', actions.review),
-    re_path(r'^quotate/?$', actions.quotate),
+    re_path(r'^quote/?$', actions.quotate),
     re_path(r'^comment/?$', actions.comment),
     re_path(r'^tag/?$', actions.tag),
     re_path(r'^untag/?$', actions.untag),

bookwyrm/views.py

@@ -430,7 +430,7 @@ def status_page(request, username, status_id):
         return JsonResponse(status.to_activity(), encoder=ActivityEncoder)
 
     data = {
-        'title': status.type,
+        'title': 'Status by %s' % user.username,
         'status': status,
     }
     return TemplateResponse(request, 'status.html', data)