Minor UI tweaks and fixes

diff --git a/src/littlesongplace/activity.py b/src/littlesongplace/activity.py
index eab6d3a0a6a11603173024eb18650da0a5e29364..e3df8d4ce9b2aa56f95609ccea3826e521cd587c 100644 (file)
--- a/src/littlesongplace/activity.py
+++ b/src/littlesongplace/activity.py
@@ -46,6 +46,8 @@ def activity():
                     "select * from users where threadid = ?",
                     [comment["threadid"]],
                     one=True)
+            if profile is None:
+                continue
             comment["content_userid"] = profile["userid"]
             comment["content_username"] = profile["username"]
         elif threadtype == comments.ThreadType.PLAYLIST:
@@ -58,6 +60,8 @@ def activity():
                 [comment["threadid"]],
                 one=True,
             )
+            if playlist is None:
+                continue
             comment["playlistid"] = playlist["playlistid"]
             comment["name"] = playlist["name"]
             comment["content_userid"] = playlist["userid"]
@@ -70,6 +74,8 @@ def activity():
                     INNER JOIN users ON jams.ownerid = users.userid
                     WHERE jam_events.threadid = ?
                     """, [comment["threadid"]], one=True)
+            if jam_event is None:
+                continue
             # TODO: This is duplicated in the JamEvent class
             startdate = datetime.fromisoformat(jam_event["startdate"]) if jam_event["startdate"] else None
             hidden = ((startdate is None) or startdate > datetime.now(timezone.utc))

diff --git a/src/littlesongplace/static/styles.css b/src/littlesongplace/static/styles.css
index 2133bd3ea1f7878b37e5d60dde657bcee53c01c4..a907b003e9440de650cece4fa372a5747bb75797 100644 (file)
--- a/src/littlesongplace/static/styles.css
+++ b/src/littlesongplace/static/styles.css
@@ -336,7 +336,8 @@ input[type=file] {
 }
 
 .draggable-song {
-    box-shadow: 0px 0px 5px 0px;
+    /*box-shadow: 0px 0px 5px 0px;*/
+    border: 2px solid;
     border-radius: var(--radius);
     padding: 5px 10px;
     margin: 10px 0px;
@@ -465,7 +466,7 @@ div.song-details {
 div.top-level-comment {
     margin-top: 10px;
     padding: 10px;
-    box-shadow: 0px 0px 5px 0px;
+    border: 2px solid;
     border-radius: var(--radius);
 }
 
@@ -473,7 +474,7 @@ div.reply-comment {
     margin-top: 10px;
     margin-bottom: 10px;
     padding: 10px;
-    box-shadow: 0px 0px 5px 0px;
+    border: 2px solid;
     border-radius: var(--radius);
 }
 
@@ -505,7 +506,8 @@ div.player {
     margin: 10px;
     padding: 10px;
     padding-bottom: 0px;
-    box-shadow: 0px 0px 20px 0px;
+    box-shadow: 0px 0px 10px 0px;
+    border: 2px solid;
     border-radius: var(--radius);
     background: var(--yellow);
 }
@@ -513,7 +515,8 @@ div.player {
 .mini-player {
     margin: 10px;
     padding-left: 10px;
-    box-shadow: 0px 0px 20px 0px;
+    box-shadow: 0px 0px 10px 0px;
+    border: 2px solid;
     border-radius: var(--radius);
     background: var(--yellow);
 
@@ -613,7 +616,8 @@ div.player-info {
 div.comment-notification {
     margin: 10px;
     padding: 10px;
-    box-shadow: 0px 0px 5px 0px;
+    box-shadow: 2px 2px 0px 0px;
+    border: 1px solid;
     border-radius: var(--radius);
 }