ticket #94 attempt 1

devplacepy/static/css/feed.css

@@ -222,8 +222,6 @@
     display: flex;
     flex-direction: column;
     gap: 1rem;
-    max-height: calc(100vh - var(--nav-height) - 2rem);
-    overflow-y: auto;
 }
 
 .daily-topic-card {

devplacepy/static/css/sidebar.css

@@ -8,8 +8,6 @@
     border-radius: var(--radius-lg);
     box-shadow: var(--shadow-sm);
     padding: 1rem;
-    max-height: calc(100vh - var(--nav-height) - 2rem);
-    overflow-y: auto;
 }
 
 .sidebar-card .sidebar-heading {

devplacepy/static/js/VoteManager.js

@@ -5,18 +5,21 @@ import { OptimisticAction } from "./OptimisticAction.js";
 export class VoteManager extends OptimisticAction {
     constructor() {
         super();
-        this.initVoteButtons();
+        document.addEventListener("click", (event) => this.onClick(event));
     }
 
-    initVoteButtons() {
-        document.querySelectorAll('form[action^="/votes/"] button[type="submit"]').forEach((button) => {
-            const form = button.closest("form");
-            button.addEventListener("click", (event) => {
-                event.preventDefault();
-                event.stopPropagation();
-                this.cast(form, button);
-            });
-        });
+    onClick(event) {
+        const button = event.target.closest('form[action^="/votes/"] button[type="submit"]');
+        if (!button) {
+            return;
+        }
+        const form = button.closest("form");
+        if (!form) {
+            return;
+        }
+        event.preventDefault();
+        event.stopPropagation();
+        this.cast(form, button);
     }
 
     cast(form, button) {

devplacepy/templating.py

@@ -29,7 +29,7 @@ def is_self(user, uid) -> bool:
 def guest_disabled(user) -> Markup:
     if user:
         return Markup("")
-    return Markup(' disabled aria-disabled="true"')
+    return Markup(' disabled aria-disabled="true" title="Log in to vote"')
 
 
 def static_url(path) -> str: