From da30b0240545f1269799289df8cbb76534136261 Mon Sep 17 00:00:00 2001
From: ricola <ricola@poivron.org>
Date: Fri, 9 Jan 2026 15:35:33 -0600
Subject: [PATCH 1/8] Add feedback when voting

---
 vedia.rb                  | 4 ++--
 views/votes.erb           | 5 +++++
 views/votes_show_open.erb | 9 +++++++++
 3 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/vedia.rb b/vedia.rb
index 34c37dd..0fd3cf7 100644
--- a/vedia.rb
+++ b/vedia.rb
@@ -510,15 +510,15 @@ post '/votes/:id/ratings' do
   end
   if not @errors.empty?
     @params = params
-    erb :votes_show_open
   else
     @vote.candidates.each do |candidate|
       rating = Rating.find_or_initialize_by(vote: @vote, user: current_user, candidate: candidate)
       rating.value = params[candidate.id.to_s]
       rating.save
     end
-    redirect '/votes/' + @vote.secure_id
+    @voted = true
   end
+  erb :votes_show_open
 end
 
 get '/votes/:id/organizers' do
diff --git a/views/votes.erb b/views/votes.erb
index fe1f7a7..1e10aed 100644
--- a/views/votes.erb
+++ b/views/votes.erb
@@ -14,6 +14,11 @@
           <% when 'closed' %>
             <SPAN class="badge bg-dark"><%= _("Closed") %></span>
           <% end %>
+          <% if vote.ratings.find { |rating| rating.user == current_user } %>
+            <span class="badge bg-success"><%= _("Voted") %></span>
+          <% elsif vote.state == 'open' %>
+            <span class="badge bg-danger"><%= _("Not voted") %></span>
+          <% end %>
           <% if vote.users.exists?(current_user.id) %>
             <span class="badge bg-warning text-bg-warning"><%= _("Organizer") %></span>
           <% end %>
diff --git a/views/votes_show_open.erb b/views/votes_show_open.erb
index 1b8cf8d..f69b1bc 100644
--- a/views/votes_show_open.erb
+++ b/views/votes_show_open.erb
@@ -3,6 +3,11 @@
   <ul class="list-unstyled">
     <li>
       <span class="badge bg-primary"><%= _("Open") %></span>
+      <% if @vote.ratings.find { |rating| rating.user == current_user } %>
+        <span class="badge bg-success"><%= _("Voted") %></span>
+      <% else %>
+        <span class="badge bg-danger"><%= _("Not voted") %></span>
+      <% end %>
     </li>
     <% if @vote.expire_on %>
       <li><%= _("Closes on %{date}") % { date: format_date_and_time(@vote.expire_on) } %></li>
@@ -26,6 +31,10 @@
 <% end %>
 <% end %>
 
+<% if @voted %>
+  <p class="alert alert-success mb-4"><%= _("Gràcies per votar!") %></p>
+<% end %>
+
 <form action="/votes/<%= @vote.secure_id %>/ratings" method="post" class="mb-5">
 
   <% @vote.candidates.each do |candidate| %>

From 876d46bc67460c5c8a48ecc2e3323fd3956b67bb Mon Sep 17 00:00:00 2001
From: ricola <ricola@poivron.org>
Date: Fri, 9 Jan 2026 15:46:32 -0600
Subject: [PATCH 2/8] Sort list of users alphabetically

---
 views/admin.erb             | 2 +-
 views/admin_votes.erb       | 2 +-
 views/votes_show_closed.erb | 2 +-
 views/votes_show_open.erb   | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/views/admin.erb b/views/admin.erb
index 802627c..9857d75 100644
--- a/views/admin.erb
+++ b/views/admin.erb
@@ -3,7 +3,7 @@
 <h2 class="mb-4"><%= _("Users") %></h2>
 
 <ul class="mb-5">
-<% @users.reverse.each do |user| %>
+<% @users.sort_by { |user| user.email }.each do |user| %>
   <li>
   <a href="/admin/users/<%= user.id %>"><%= user.email %></a>
   </li>
diff --git a/views/admin_votes.erb b/views/admin_votes.erb
index e5a443a..b580872 100644
--- a/views/admin_votes.erb
+++ b/views/admin_votes.erb
@@ -40,7 +40,7 @@
 <% unless @vote.ratings.blank? %>
 
 <ul>
-<% @vote.ratings.collect { |rating| rating.user }.uniq.each do |user| %>
+<% @vote.ratings.collect { |rating| rating.user }.uniq.sort_by { |user| user.email }.each do |user| %>
   <li>
     <%= user.email %>
     <a href="/admin/votes/<%= @vote.id %>/ratings/<%= user.id %>/delete"><%= _("Delete") %></a>
diff --git a/views/votes_show_closed.erb b/views/votes_show_closed.erb
index d908383..4ad63eb 100644
--- a/views/votes_show_closed.erb
+++ b/views/votes_show_closed.erb
@@ -97,7 +97,7 @@ end
       <% end %>
     </tr>
   </thead>
-  <% @vote.ratings.collect { |rating| rating.user }.uniq.each do |user| %>
+  <% @vote.ratings.collect { |rating| rating.user }.uniq.sort_by { |user| user.email }.each do |user| %>
   <tr>
     <td><%= user.email %></td>
     <% @vote.candidates.each do |candidate| %>
diff --git a/views/votes_show_open.erb b/views/votes_show_open.erb
index f69b1bc..5d58d3a 100644
--- a/views/votes_show_open.erb
+++ b/views/votes_show_open.erb
@@ -66,7 +66,7 @@
 <h2 class="mb-4"><%= _("Participants") %></h2>
 
 <ul class="mb-5">
-<% @vote.ratings.collect { |rating| rating.user }.uniq.each do |user| %>
+<% @vote.ratings.collect { |rating| rating.user }.uniq.sort_by { |user| user.email }.each do |user| %>
   <li><%= user.email %></li>
 <% end %>
 </ul>

From ec173f450d03ff4938bd750917f8865719fb0c43 Mon Sep 17 00:00:00 2001
From: ricola <ricola@poivron.org>
Date: Fri, 9 Jan 2026 15:53:57 -0600
Subject: [PATCH 3/8] Add participant count

---
 views/votes_show_open.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/views/votes_show_open.erb b/views/votes_show_open.erb
index 5d58d3a..c202823 100644
--- a/views/votes_show_open.erb
+++ b/views/votes_show_open.erb
@@ -63,7 +63,7 @@
   <button type="submit" class="btn btn-primary"><%= _("Vote") %></button>
 </form>
 
-<h2 class="mb-4"><%= _("Participants") %></h2>
+<h2 class="mb-4"><%= _("Participants") + " (#{@vote.ratings.collect { |rating| rating.user }.uniq.count})" %></h2>
 
 <ul class="mb-5">
 <% @vote.ratings.collect { |rating| rating.user }.uniq.sort_by { |user| user.email }.each do |user| %>

From 33bc3e1446db3afa641cb5178c56c617aa808dab Mon Sep 17 00:00:00 2001
From: ricola <ricola@poivron.org>
Date: Fri, 9 Jan 2026 16:05:03 -0600
Subject: [PATCH 4/8] Add 'Vote not found'

---
 vedia.rb                     | 24 ++++++++++++++----------
 views/votes_show_unknown.erb |  3 +++
 2 files changed, 17 insertions(+), 10 deletions(-)
 create mode 100644 views/votes_show_unknown.erb

diff --git a/vedia.rb b/vedia.rb
index 0fd3cf7..4c18ff2 100644
--- a/vedia.rb
+++ b/vedia.rb
@@ -319,17 +319,21 @@ end
 get '/votes/:id' do
   require_login
   find_vote
-  case @vote.state
-  when 'draft'
-    if @vote.users.exists?(current_user.id)
-      erb :votes_edit
-    else
-      erb :votes_show_draft
+  if not @vote
+    erb :votes_show_unknown
+  else
+    case @vote.state
+    when 'draft'
+      if @vote.users.exists?(current_user.id)
+        erb :votes_edit
+      else
+        erb :votes_show_draft
+      end
+    when 'open'
+      erb :votes_show_open
+    when 'closed'
+      erb :votes_show_closed
     end
-  when 'open'
-    erb :votes_show_open
-  when 'closed'
-    erb :votes_show_closed
   end
 end
 
diff --git a/views/votes_show_unknown.erb b/views/votes_show_unknown.erb
new file mode 100644
index 0000000..0d35540
--- /dev/null
+++ b/views/votes_show_unknown.erb
@@ -0,0 +1,3 @@
+<div class="mb-5">
+  <h1><%= _("Vote not found...") %></h1>
+</div>

From 45c8e9b6b472e0087ca5a6477d440fdad1458b51 Mon Sep 17 00:00:00 2001
From: ricola <ricola@poivron.org>
Date: Fri, 9 Jan 2026 16:25:10 -0600
Subject: [PATCH 5/8] Display vote name even when unauthenticated

For example, when sharing the link in messaging apps.
---
 vedia.rb                             | 25 ++++++++++++++-----------
 views/layout.erb                     |  2 ++
 views/votes_show_unauthenticated.erb | 17 +++++++++++++++++
 3 files changed, 33 insertions(+), 11 deletions(-)
 create mode 100644 views/votes_show_unauthenticated.erb

diff --git a/vedia.rb b/vedia.rb
index 4c18ff2..2ca3c3a 100644
--- a/vedia.rb
+++ b/vedia.rb
@@ -317,22 +317,25 @@ post '/votes/new' do
 end
 
 get '/votes/:id' do
-  require_login
   find_vote
   if not @vote
     erb :votes_show_unknown
   else
-    case @vote.state
-    when 'draft'
-      if @vote.users.exists?(current_user.id)
-        erb :votes_edit
-      else
-        erb :votes_show_draft
+    if not current_user
+      erb :votes_show_unauthenticated
+    else
+      case @vote.state
+      when 'draft'
+        if @vote.users.exists?(current_user.id)
+          erb :votes_edit
+        else
+          erb :votes_show_draft
+        end
+      when 'open'
+        erb :votes_show_open
+      when 'closed'
+        erb :votes_show_closed
       end
-    when 'open'
-      erb :votes_show_open
-    when 'closed'
-      erb :votes_show_closed
     end
   end
 end
diff --git a/views/layout.erb b/views/layout.erb
index 5b55f67..c7c3d09 100644
--- a/views/layout.erb
+++ b/views/layout.erb
@@ -46,6 +46,8 @@
       <div class="d-flex flex-wrap justify-content-end pe-3 py-3 mb-4">
         <% if current_user %>
           <%= current_user.email %>
+        <% else %>
+          &nbsp;
         <% end %>
       </div>
 
diff --git a/views/votes_show_unauthenticated.erb b/views/votes_show_unauthenticated.erb
new file mode 100644
index 0000000..6899784
--- /dev/null
+++ b/views/votes_show_unauthenticated.erb
@@ -0,0 +1,17 @@
+<div class="mb-5">
+  <h1><%= @vote.title %></h1>
+</div>
+
+<p class="alert alert-warning mb-4"><%= _("You need to log in to see the details of this vote.") %></p>
+
+<form action="/login" method="post" class="mb-5">
+  <div class="mb-3">
+    <label for="email" class="form-label"><%= _("Email") %></label>
+    <input type="text" name="email" class="form-control">
+  </div>
+  <div class="mb-3">
+    <label for="password" class="form-label"><%= _("Password") %></label>
+    <input type="password" name="password" class="form-control">
+  </div>
+  <button type="submit" class="btn btn-primary"><%= _("Login") %></button>
+</form>

From d2e07630d5804ddcc2745b17d3cfe7443b500ab8 Mon Sep 17 00:00:00 2001
From: ricola <ricola@poivron.org>
Date: Fri, 9 Jan 2026 16:25:56 -0600
Subject: [PATCH 6/8] Display vote title in page title and when sharing link

---
 views/layout.erb | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/views/layout.erb b/views/layout.erb
index c7c3d09..2ccbbd8 100644
--- a/views/layout.erb
+++ b/views/layout.erb
@@ -3,7 +3,12 @@
   <head>
     <meta charset="utf-8">
     <meta name="viewport" content="width=device-width, initial-scale=1">
-    <title>Vedia</title>
+    <% if @vote %>
+      <title>Vedia · <%= @vote.title %></title>
+      <meta property="og:title" content="<%= @vote.title %>"/>
+    <% else %>
+      <title>Vedia</title>
+    <% end %>
     <script src="/chartkick.js"></script>
     <script src="/Chart.bundle.js"></script>
     <link rel="stylesheet" href="/bootstrap.css">

From d55dc53ade8adf36054c00273a0224391701af7e Mon Sep 17 00:00:00 2001
From: ricola <ricola@poivron.org>
Date: Fri, 9 Jan 2026 17:04:14 -0600
Subject: [PATCH 7/8] Set time zone on login

Should be more reliable.
---
 vedia.rb                             | 12 ++----------
 views/layout.erb                     |  9 ---------
 views/login.erb                      |  7 ++++++-
 views/votes_show_unauthenticated.erb |  5 +++++
 4 files changed, 13 insertions(+), 20 deletions(-)

diff --git a/vedia.rb b/vedia.rb
index 2ca3c3a..e641ded 100644
--- a/vedia.rb
+++ b/vedia.rb
@@ -133,6 +133,7 @@ post '/login' do
     end
     session.clear
     session[:user_id] = user.id
+    session[:timezone] = params[:timezone]
     redirect '/'
   else
     @error = _("Incorrect email or password.")
@@ -140,10 +141,6 @@ post '/login' do
   end
 end
 
-post '/timezone' do
-  session[:timezone] = JSON.parse(request.body.read)['timezone']
-end
-
 get '/reset' do
   erb :reset
 end
@@ -641,12 +638,7 @@ helpers do
   end
 
   def format_date(timestamp)
-    if session[:timezone]
-      "#{TZInfo::Timezone.get(session[:timezone]).to_local(timestamp).strftime('%F')}"
-    else
-      # Otherwise, format_date fails on first page after login. Not sure why...
-      "#{timestamp.strftime('%F')}"
-    end
+    "#{TZInfo::Timezone.get(session[:timezone]).to_local(timestamp).strftime('%F')}"
   end
 
   def format_date_and_time(timestamp)
diff --git a/views/layout.erb b/views/layout.erb
index 2ccbbd8..a5dbe8d 100644
--- a/views/layout.erb
+++ b/views/layout.erb
@@ -15,15 +15,6 @@
     <link rel="stylesheet" href="/bootstrap-icons.css">
     <link rel="stylesheet" href="/style.css">
   </head>
-  <script>
-  fetch('/timezone', {
-    method: 'post',
-    headers: {
-        'Content-Type': 'application/json'
-    },
-    body: JSON.stringify({ timezone: Intl.DateTimeFormat().resolvedOptions().timeZone })
-  });
-  </script>
   <body>
     <div class="container py-4">
 
diff --git a/views/login.erb b/views/login.erb
index 5a5f40c..5af0607 100644
--- a/views/login.erb
+++ b/views/login.erb
@@ -4,7 +4,7 @@
   <p class="alert alert-warning mb-4"><%= @error %></p>
 <% end %>
 
-<form action="/login" method="post" class="mb-5">
+<form action="/login" name="login" method="post" class="mb-5">
   <div class="mb-3">
     <label for="email" class="form-label"><%= _("Email") %></label>
     <input type="text" name="email" class="form-control">
@@ -14,6 +14,11 @@
     <input type="password" name="password" class="form-control">
   </div>
   <button type="submit" class="btn btn-primary"><%= _("Login") %></button>
+  <input type="hidden" name="timezone" value="UTC">
 </form>
 <p><a href="/signup"><%= _("Create account") %></a></p>
 <p><a href="/reset"><%= _("Reset password") %></a></p>
+
+<script>
+document.login.timezone.value = Intl.DateTimeFormat().resolvedOptions().timeZone;
+</script>
diff --git a/views/votes_show_unauthenticated.erb b/views/votes_show_unauthenticated.erb
index 6899784..2d43efe 100644
--- a/views/votes_show_unauthenticated.erb
+++ b/views/votes_show_unauthenticated.erb
@@ -14,4 +14,9 @@
     <input type="password" name="password" class="form-control">
   </div>
   <button type="submit" class="btn btn-primary"><%= _("Login") %></button>
+  <input type="hidden" name="timezone" value="UTC">
 </form>
+
+<script>
+document.login.timezone.value = Intl.DateTimeFormat().resolvedOptions().timeZone;
+</script>

From 38a37137d004c1c2512c7f917850fd2a29bb5a97 Mon Sep 17 00:00:00 2001
From: ricola <ricola@poivron.org>
Date: Fri, 9 Jan 2026 17:19:18 -0600
Subject: [PATCH 8/8] Redirect to previous page after login

---
 vedia.rb                             | 9 +++++++--
 views/login.erb                      | 1 +
 views/votes_show_unauthenticated.erb | 1 +
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/vedia.rb b/vedia.rb
index e641ded..4b6e308 100644
--- a/vedia.rb
+++ b/vedia.rb
@@ -134,7 +134,11 @@ post '/login' do
     session.clear
     session[:user_id] = user.id
     session[:timezone] = params[:timezone]
-    redirect '/'
+    if params[:r]
+      redirect params[:r]
+    else
+      redirect '/'
+    end
   else
     @error = _("Incorrect email or password.")
     erb :login
@@ -598,10 +602,11 @@ helpers do
   end
 
   def require_login
-    redirect '/login' unless current_user
+    redirect "/login?r=#{request.path}" unless current_user
   end
 
   def require_admin
+    require_login
     redirect '/' unless is_admin
   end
 
diff --git a/views/login.erb b/views/login.erb
index 5af0607..9edb123 100644
--- a/views/login.erb
+++ b/views/login.erb
@@ -14,6 +14,7 @@
     <input type="password" name="password" class="form-control">
   </div>
   <button type="submit" class="btn btn-primary"><%= _("Login") %></button>
+  <input type="hidden" name="r" value="<%= @params[:r] %>">
   <input type="hidden" name="timezone" value="UTC">
 </form>
 <p><a href="/signup"><%= _("Create account") %></a></p>
diff --git a/views/votes_show_unauthenticated.erb b/views/votes_show_unauthenticated.erb
index 2d43efe..e6dab00 100644
--- a/views/votes_show_unauthenticated.erb
+++ b/views/votes_show_unauthenticated.erb
@@ -14,6 +14,7 @@
     <input type="password" name="password" class="form-control">
   </div>
   <button type="submit" class="btn btn-primary"><%= _("Login") %></button>
+  <input type="hidden" name="r" value="votes/<%= @vote.secure_id %>">
   <input type="hidden" name="timezone" value="UTC">
 </form>