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 35b93f33c45c437aae60654ffe139b789c3d47a4..be9669a1911c60fa0840593dc802dcc9af7bbd28 100644 (file)
--- a/src/littlesongplace/static/styles.css
+++ b/src/littlesongplace/static/styles.css
@@ -412,7 +412,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;
@@ -538,7 +539,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);
 }
 
@@ -546,7 +547,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);
 }
 
@@ -578,7 +579,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);
 }
@@ -586,7 +588,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);
 
@@ -686,7 +689,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);
 }