From 311e13f696363d995325bd1fad8c040b5400d728 Mon Sep 17 00:00:00 2001
From: William Kuijltjes <wkuijltjes@science.ru.nl>
Date: Mon, 18 Mar 2024 12:45:15 +0100
Subject: [PATCH 1/3] Fix toggling of Plays In/Plays At in mobile view

---
 marietje/queues/templates/queues/queue.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/marietje/queues/templates/queues/queue.html b/marietje/queues/templates/queues/queue.html
index e38035e..98fa5b6 100644
--- a/marietje/queues/templates/queues/queue.html
+++ b/marietje/queues/templates/queues/queue.html
@@ -78,13 +78,13 @@
                             <td class="col-md-4">Title</td>
                             <td class="col-md-2 d-sm-table-cell d-none">Requested By</td>
                             <td class="col-md-1 text-info d-sm-table-cell d-none" style="cursor: pointer;">
-                                <span v-if="playsIn" class="btn btn-link p-0" v-on:click="playsIn = false">Plays In</span>
-                                <span v-else class="btn btn-link p-0" v-on:click="playsIn = true">Plays At</span>
+                                <span v-if="playsIn" class="btn btn-link p-0 text-decoration-none" v-on:click="playsIn = false">Plays In</span>
+                                <span v-else class="btn btn-link p-0 text-decoration-none" v-on:click="playsIn = true">Plays At</span>
                             </td>
                             <td class="col-md-1">
                                 <span class="control-icons">Control</span>
-                                <span v-if="playsIn" class="btn btn-link p-0 d-sm-none" v-on:click="toggle_details(song)">(Plays in)</span>
-                                <span v-else class="btn btn-link p-0 d-sm-none" v-on:click="toggle_details(song)">(Plays At)</span>
+                                <span v-if="playsIn" class="btn btn-link p-0 text-decoration-none d-sm-none" v-on:click="playsIn = false" v-on:click="toggle_details(song)">(Plays In)</span>
+                                <span v-else class="btn btn-link p-0 text-decoration-none d-sm-none" v-on:click="playsIn = true" v-on:click="toggle_details(song)">(Plays At)</span>
                             </td>
                         </tr>
                         </thead>

From 3c2df2adfd0d04fe666216a2f620a512f89fc85d Mon Sep 17 00:00:00 2001
From: Kees van Kempen <k.vankempen@student.science.ru.nl>
Date: Tue, 23 Apr 2024 17:56:58 +0200
Subject: [PATCH 2/3] styling(queues): move no underline to css

---
 marietje/marietje/static/marietje/css/custom.css |  4 ++++
 marietje/queues/templates/queues/queue.html      | 12 ++++++------
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/marietje/marietje/static/marietje/css/custom.css b/marietje/marietje/static/marietje/css/custom.css
index e5e7055..e75d42a 100644
--- a/marietje/marietje/static/marietje/css/custom.css
+++ b/marietje/marietje/static/marietje/css/custom.css
@@ -105,6 +105,10 @@ footer {
     transition: 1s transform ease-in-out;
 }
 
+.btn-link {
+    text-decoration: none;
+}
+
 .navbar-text {
     color: var(--text-color);
 }
diff --git a/marietje/queues/templates/queues/queue.html b/marietje/queues/templates/queues/queue.html
index 98fa5b6..abe5e9d 100644
--- a/marietje/queues/templates/queues/queue.html
+++ b/marietje/queues/templates/queues/queue.html
@@ -78,13 +78,13 @@
                             <td class="col-md-4">Title</td>
                             <td class="col-md-2 d-sm-table-cell d-none">Requested By</td>
                             <td class="col-md-1 text-info d-sm-table-cell d-none" style="cursor: pointer;">
-                                <span v-if="playsIn" class="btn btn-link p-0 text-decoration-none" v-on:click="playsIn = false">Plays In</span>
-                                <span v-else class="btn btn-link p-0 text-decoration-none" v-on:click="playsIn = true">Plays At</span>
+                                <span v-if="playsIn" class="btn btn-link p-0" v-on:click="playsIn = false">Plays In</span>
+                                <span v-else class="btn btn-link p-0" v-on:click="playsIn = true">Plays At</span>
                             </td>
                             <td class="col-md-1">
                                 <span class="control-icons">Control</span>
-                                <span v-if="playsIn" class="btn btn-link p-0 text-decoration-none d-sm-none" v-on:click="playsIn = false" v-on:click="toggle_details(song)">(Plays In)</span>
-                                <span v-else class="btn btn-link p-0 text-decoration-none d-sm-none" v-on:click="playsIn = true" v-on:click="toggle_details(song)">(Plays At)</span>
+                                <span v-if="playsIn" class="btn btn-link p-0 d-sm-none" v-on:click="playsIn = false" v-on:click="toggle_details(song)">(Plays In)</span>
+                                <span v-else class="btn btn-link p-0 d-sm-none" v-on:click="playsIn = true" v-on:click="toggle_details(song)">(Plays At)</span>
                             </td>
                         </tr>
                         </thead>
@@ -224,7 +224,7 @@
                                     ${ song.artist }$
                                 </td>
                                 <td>
-                                    <button v-on:click="request_song(song.id);" class="btn btn-link p-0 text-decoration-none" style="text-align: left">${ song.title }$</button>
+                                    <button v-on:click="request_song(song.id);" class="btn btn-link p-0" style="text-align: left">${ song.title }$</button>
                                 </td>
                                 <td>
                                     <template v-if="song.user === null">
@@ -238,7 +238,7 @@
                                     ${ song.duration.secondsToMMSS() }$
                                 </td>
                                 <td>
-                                    <button v-on:click="report_song(song.id);" class="btn btn-link p-0 text-decoration-none">
+                                    <button v-on:click="report_song(song.id);" class="btn btn-link p-0">
                                         ⚑
                                     </button>
                                 </td>

From 7ab6bc6c78adb26ad812b950248f577fa75a0008 Mon Sep 17 00:00:00 2001
From: Kees van Kempen <k.vankempen@student.science.ru.nl>
Date: Tue, 23 Apr 2024 18:11:59 +0200
Subject: [PATCH 3/3] fix(queues): remove redundant and faulty toggle_details
 call

song is not even defined in this scope.
---
 marietje/queues/templates/queues/queue.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/marietje/queues/templates/queues/queue.html b/marietje/queues/templates/queues/queue.html
index abe5e9d..fb42948 100644
--- a/marietje/queues/templates/queues/queue.html
+++ b/marietje/queues/templates/queues/queue.html
@@ -83,8 +83,8 @@
                             </td>
                             <td class="col-md-1">
                                 <span class="control-icons">Control</span>
-                                <span v-if="playsIn" class="btn btn-link p-0 d-sm-none" v-on:click="playsIn = false" v-on:click="toggle_details(song)">(Plays In)</span>
-                                <span v-else class="btn btn-link p-0 d-sm-none" v-on:click="playsIn = true" v-on:click="toggle_details(song)">(Plays At)</span>
+                                <span v-if="playsIn" class="btn btn-link p-0 d-sm-none" v-on:click="playsIn = false" >(Plays In)</span>
+                                <span v-else class="btn btn-link p-0 d-sm-none" v-on:click="playsIn = true">(Plays At)</span>
                             </td>
                         </tr>
                         </thead>