From: Chris Fulljames <christianfulljames@gmail.com>
Date: Tue, 11 Feb 2025 01:46:17 +0000 (-0500)
Subject: Revert "Use song colors in song lists outside profile"
X-Git-Url: https://littlesong.place/gitweb/?a=commitdiff_plain;h=14343f16acdb034c8c3cfabe274d30c7e548249b;p=littlesongplace.git

Revert "Use song colors in song lists outside profile"

This reverts commit 5652f48ccb4109e188eb497f1c6d8ca6b132820c.
---

diff --git a/main.py b/main.py
index 72ef925..d777b48 100644
--- a/main.py
+++ b/main.py
@@ -802,9 +802,6 @@ class Song:
     description: str
     tags: list[str]
     collaborators: list[str]
-    fgcolor: str
-    bgcolor: str
-    accolor: str
 
     def json(self):
         return json.dumps(vars(self))
@@ -860,18 +857,7 @@ class Song:
         for sd in songs_data:
             song_tags = [t["tag"] for t in tags[sd["songid"]]]
             song_collabs = [c["name"] for c in collabs[sd["songid"]]]
-            songs.append(cls(
-                sd["songid"],
-                sd["userid"],
-                sd["username"],
-                sd["title"],
-                sanitize_user_text(sd["description"]),
-                song_tags,
-                song_collabs,
-                sd["fgcolor"],
-                sd["bgcolor"],
-                sd["accolor"],
-            ))
+            songs.append(cls(sd["songid"], sd["userid"], sd["username"], sd["title"], sanitize_user_text(sd["description"]), song_tags, song_collabs))
 
         return songs
 
diff --git a/static/player.js b/static/player.js
index ab0327e..868e8dd 100644
--- a/static/player.js
+++ b/static/player.js
@@ -158,13 +158,13 @@ document.addEventListener("DOMContentLoaded", (event) => {
     var button = document.getElementById("play-pause-button");
     audio.addEventListener("play", (event) => {
         button.className = "lsp_btn_pause02";
-        button.src = customImage(document.getElementById("lsp_btn_pause02"), button);
+        button.src = customImage(document.getElementById("lsp_btn_pause02"));
     })
 
     // Show play button when audio is paused
     audio.addEventListener("pause", (event) => {
         button.className = "lsp_btn_play02";
-        button.src = customImage(document.getElementById("lsp_btn_play02"), button);
+        button.src = customImage(document.getElementById("lsp_btn_play02"));
     })
 
     // Audio position scrubbing
diff --git a/static/styles.css b/static/styles.css
index 2d118ba..130bc63 100644
--- a/static/styles.css
+++ b/static/styles.css
@@ -244,7 +244,6 @@ div.song-list {
 div.song {
     box-shadow: 0px 0px 5px 0px;
     border-radius: 10px;
-    background-color: var(--yellow);
 }
 
 div.song-main {
@@ -263,10 +262,6 @@ div.song-info {
     margin: 10px;
 }
 
-div.song-info-sep {
-    color: var(--black);
-}
-
 /* Artist on separate line for mobile */
 @media screen and (max-width: 480px) {
     div.song-info {
@@ -298,10 +293,6 @@ div.song-buttons {
     width: 32px;
 }
 
-.collab-name {
-    color: var(--black);
-}
-
 div.song-details {
     display: flex;
     flex-direction: column;
diff --git a/templates/base.html b/templates/base.html
index fe17aee..693afd3 100644
--- a/templates/base.html
+++ b/templates/base.html
@@ -17,19 +17,19 @@
 
         <script>
 
-            function customImage(source, target) {
+            function customImage(element) {
                 // Customize an image by performing a palette swap on the .gif
                 // file.  The source element must contain a data-img-b64 attribute
                 // containing the base64 representation of a .gif file.  The byte
                 // indexes match .gifs from Aseprite, and may not work for all
                 // .gif files.
 
-                var style = window.getComputedStyle(target);
+                var style = window.getComputedStyle(document.body);
                 var bgcolor = style.getPropertyValue("--yellow");
                 var accolor = style.getPropertyValue("--purple");
 
                 // Convert base64 string to Uint8Array so we can modify it
-                var data = atob(source.dataset.imgB64);
+                var data = atob(element.dataset.imgB64);
                 var bytes = Uint8Array.from(data, c => c.charCodeAt(0));
 
                 // Replace background color palette bytes in gif file
@@ -52,9 +52,9 @@
             function updateImageColors() {
                 // Perform a palette swap on all gifs based on current page colors
 
-                document.querySelectorAll(".img-data").forEach(s => {
-                    document.querySelectorAll(`.${s.id}`).forEach(t => {
-                        t.src = customImage(s, t);
+                document.querySelectorAll(".img-data").forEach(e => {
+                    document.querySelectorAll(`.${e.id}`).forEach(t => {
+                        t.src = customImage(e);
                     });
                 });
             }
diff --git a/templates/song-list.html b/templates/song-list.html
index 43ab39b..7d39176 100644
--- a/templates/song-list.html
+++ b/templates/song-list.html
@@ -1,6 +1,6 @@
 <div class="song-list">
 {% for song in songs %}
-<div class="song" style="{% if song.bgcolor %}--yellow: {{ song.bgcolor }};{% endif %} {% if song.fgcolor %}--black: {{ song.fgcolor }};{% endif %} {% if song.accolor %}--purple: {{ song.accolor }};{% endif %}" data-song="{{ song.json() }}">
+    <div class="song" data-song="{{ song.json() }}">
         <div class="song-main">
             <!-- Profile Picture -->
             <img class="small-pfp" src="/pfp/{{ song.userid }}" onerror="this.style.display = 'none'" />
@@ -161,14 +161,14 @@ function showDetails(event) {
                 child.hidden = false;
                 event.target.alt = "Hide Details";
                 event.target.className = "lsp_btn_hide02";
-                event.target.src = customImage(document.getElementById("lsp_btn_hide02"), event.target);
+                event.target.src = customImage(document.getElementById("lsp_btn_hide02"));
             }
             else {
                 // Hide details
                 child.hidden = true;
                 event.target.alt = "Show Details";
                 event.target.className = "lsp_btn_show02";
-                event.target.src = customImage(document.getElementById("lsp_btn_show02"), event.target);
+                event.target.src = customImage(document.getElementById("lsp_btn_show02"));
             }
         }
     }