[boardgames] Add better list template and fix URLs (also for moods)

vrobbler/apps/boardgames/urls.py

@@ -6,10 +6,12 @@ app_name = "boardgames"
 
 urlpatterns = [
     path(
-        "board-game/", views.BoardGameListView.as_view(), name="boardgame_list"
+        "board-games/",
+        views.BoardGameListView.as_view(),
+        name="boardgame_list",
     ),
     path(
-        "board-game/<slug:slug>/",
+        "board-games/<slug:slug>/",
         views.BoardGameDetailView.as_view(),
         name="boardgame_detail",
     ),

vrobbler/apps/moods/urls.py

@@ -5,9 +5,9 @@ app_name = "moods"
 
 
 urlpatterns = [
-    path("mood/", views.MoodListView.as_view(), name="mood-list"),
+    path("moods/", views.MoodListView.as_view(), name="mood-list"),
     path(
-        "mood/<slug:slug>/",
+        "moods/<slug:slug>/",
         views.MoodDetailView.as_view(),
         name="mood-detail",
     ),

vrobbler/templates/boardgames/boardgame_list.html

@@ -1,82 +1,43 @@
 {% extends "base_list.html" %}
-{% load urlreplace %}
 
-{% block title %}Albums{% endblock %}
+{% block title %}Board Games{% endblock %}
 
-{% block lists %}
+{% block head_extra %}
+<style>
+ dl { width: 210px; float:left; margin-right: 10px; }
+ dt a { color:white; text-decoration: none; font-size:smaller; }
+ img { height:200px; width: 200px; object-fit: cover; }
+ dd .right { float:right; }
+</style>
+{% endblock  %}
 
+{% block lists %}
 <div class="row">
-    <p class="view">
-        <span class="view-links">
-            {% if view == 'grid' %}
-            View as <a href="?{% urlreplace view='list' %}">List</a>
-            {% else %}
-            View as <a href="?{% urlreplace view='grid' %}">Grid</a>
-            {% endif %}
-        </span>
-    </p>
-    <p class="pagination">
-        <span class="page-links">
-            {% if page_obj.has_previous %}
-            <a href="?{% urlreplace page=page_obj.previous_page_number %}">prev</a>
-            {% endif %}
-            <span class="page-current">
-                Page {{ page_obj.number }} of {{ page_obj.paginator.num_pages }}
-            </span>
-            {% if page_obj.has_next %}
-            <a href="?{% urlreplace page=page_obj.next_page_number %}">next</a>
-            {% endif %}
-        </span>
-    </p>
-    <hr />
 
-    {% if view == 'grid' %}
-    <div>
-        {% for game in object_list %}
-        {% if game.cover %}
-        <dl style="width: 130px; float: left; margin-right:10px;">
-            <dd><img src="{{game.cover.url}}" width=120 height=120 /></dd>
-        </dl>
-        {% endif %}
-        {% endfor %}
-    </div>
-    {% else %}
     <div class="col-md">
         <div class="table-responsive">
             <table class="table table-striped table-sm">
                 <thead>
                     <tr>
+                        <th scope="col">Title</th>
                         <th scope="col">Scrobbles</th>
-                        <th scope="col">Name</th>
-                        <th scope="col">Publisher</th>
+                        <th scope="col">Start</th>
                     </tr>
                 </thead>
                 <tbody>
-                    {% for game in object_list %}
+                    {% for obj in object_list %}
                     <tr>
-                        <td>{{game.scrobbles.count}}</td>
-                        <td><a href="{{game.get_absolute_url}}">{{game}}</a></td>
-                        <td><a href="{{game.publisher.get_absolute_url}}">{{game.publisher}}</a></td>
+                        <td><a href="{{obj.get_absolute_url}}">{{obj}}</a></td>
+                        {% if request.user.is_authenticated %}
+                        <td>{{obj.scrobble_count}}</td>
+                        <td><a type="button" class="btn btn-sm btn-primary" href="{{obj.get_start_url}}">Scrobble</a></td>
+                        {% endif %}
                     </tr>
                     {% endfor %}
                 </tbody>
             </table>
         </div>
     </div>
-    {% endif %}
-
-    <div class="pagination" style="margin-bottom:50px;">
-        <span class="page-links">
-            {% if page_obj.has_previous %}
-            <a href="?{% urlreplace page=page_obj.previous_page_number %}">prev</a>
-            {% endif %}
-            <span class="page-current">
-                Page {{ page_obj.number }} of {{ page_obj.paginator.num_pages }}
-            </span>
-            {% if page_obj.has_next %}
-            <a href="?{% urlreplace page=page_obj.next_page_number %}">next</a>
-            {% endif %}
-        </span>
-    </div>
 </div>
 {% endblock %}
+