Some minor UI improvements

Author: thordy

CHANGELOG.md

@@ -13,7 +13,7 @@ A preview of major changes can be found in the Wiki ([Latest Changes](https://gi
 #### Fixed
 - Issue with the `TTS` gets stuck in certain situations
 - Bug where matches were not correctly forwarded back to main screen
-
+- Minor UI improvements
 
 ## [2.8.0] - 2024-12-11
 #### Feature

routes/tournaments.js

@@ -461,7 +461,7 @@ router.get('/match/:id/next', function (req, res, next) {
     axios.get(`${req.app.locals.kcapp.api}/tournament/match/${req.params.id}/next`)
         .then(response => {
             if (response.status === 204) {
-                res.send(204);
+                res.sendStatus(204);
                 return;
             }
             res.send(response.data);

src/components/matches-table/components/match-row/match-row.marko

@@ -92,7 +92,9 @@ $ var columns = input.columns;
     <if(!match.has_scores)>
       <td class="text-center">
         <match-button title="View" icon="fa-cog" modal=`set-score-modal` data=match.id on-show-modal("onShowModal")/>
-        <match-button title="Start Remote" icon="fa-fast-forward" modal='start-remote-modal' data=match.id on-show-modal("onShowModal")/>
+        <if(!match.is_finished)>
+          <match-button title="Start Remote" icon="fa-fast-forward" modal='start-remote-modal' data=match.id on-show-modal("onShowModal")/>
+        </if>
       </td>
     </if>
     <else>

src/pages/match-result/components/match-result/match-result.marko

@@ -1,6 +1,6 @@
 $ let match = state.match;
 $ let isOfficial = state.match.tournament_id || false;
-<page-header heading=`Match Result (${match.id})` style="block-container" rematch_button=!isOfficial next_match_button=isOfficial subheadings=[
+<page-header heading=`Match Result (${match.id})` style="block-container" rematch_button=!isOfficial next_match_button=(isOfficial && !state.match.tournament.is_finished) subheadings=[
   [
     {
       heading: `Started   ${match.started}`,

src/pages/tournament/components/tournament/components/options-panel/options-panel.marko

@@ -1,14 +1,16 @@
-<div class="block-container-header">
-  <i class="fas fa-arrow-alt-circle-down"/>
-  <span class="ml-10">Options</span>
-</div>
-<div class="block-container-with-header">
-  <button class="btn btn-primary" type="button" data-toggle="modal" data-target="#add-player-modal">
-    <i class="fas fa-user-plus"/>
-    <span class="ml-10">Add Player</span>
-  </button>
-  <button class="btn btn-primary ml-10" type="button" data-toggle="modal" data-target="#generate-playoffs-modal">
-    <i class="fas fa-plus-square"/>
-    <span class="ml-10">Generate Playoffs</span>
-  </button>
-</div>
+<if(!input.tournament.playoffs_tournament_id)>
+  <div class="block-container-header">
+    <i class="fas fa-arrow-alt-circle-down"/>
+    <span class="ml-10">Options</span>
+  </div>
+  <div class="block-container-with-header">
+      <button class="btn btn-primary" type="button" data-toggle="modal" data-target="#add-player-modal">
+        <i class="fas fa-user-plus"/>
+        <span class="ml-10">Add Player</span>
+      </button>
+      <button class="btn btn-primary ml-10" type="button" data-toggle="modal" data-target="#generate-playoffs-modal">
+        <i class="fas fa-plus-square"/>
+        <span class="ml-10">Generate Playoffs</span>
+      </button>
+  </div>
+</if>