From b3a7c027f55b280df7b3dd8899754b61da81a66e Mon Sep 17 00:00:00 2001
From: Aidan Cornelius-Bell <aidan@cornelius-bell.com>
Date: Mon, 13 Jan 2025 13:46:00 +1030
Subject: [PATCH] Mail changes, stylesheet updates for forms, bug fixes mostly

---
 app/assets/stylesheets/application.css        | 75 +++++++++++++++++--
 app/jobs/weekly_bookmarks_digest_job.rb       |  2 +-
 app/mailers/digest_mailer.rb                  |  2 +-
 .../weekly_bookmarks_digest.html.erb          |  4 +-
 4 files changed, 74 insertions(+), 9 deletions(-)

diff --git a/app/assets/stylesheets/application.css b/app/assets/stylesheets/application.css
index 16b3c76..36f14c2 100644
--- a/app/assets/stylesheets/application.css
+++ b/app/assets/stylesheets/application.css
@@ -107,9 +107,9 @@
 
     --standout-bg: #191724;
     --standout-text: #fff;
-    --standout-form-bg: #fdf9eb;
-    --standout-form-text: #fff;
-    --standout-form-button-text: #191724;
+    --standout-form-bg: #333333;
+    --standout-form-text: #ffffff;
+    --standout-form-button-text: #ffffff;
 }
 
 /* Light mode */
@@ -135,8 +135,9 @@
         --standout-bg: #ffeee2;
         --standout-form-text: #333333;
         --standout-text: #333333;
-        --standout-form-bg: #fff;
-        --standout-form-button-text: #fff;
+        --standout-form-bg: #ffffff;
+        --standout-form-text: #333333;
+        --standout-form-button-text: #ffffff;
     }
 }
 
@@ -231,6 +232,8 @@ select {
     border-radius: 4px;
     box-sizing: border-box;
     font-size: 16px;
+    background-color: var(--standout-form-bg);
+    color: var(--standout-form-text);
 }
 
 input[type="text"]:focus,
@@ -269,6 +272,9 @@ label {
 .reg-form input {
     background-color: var(--standout-form-bg) !important;
     color: var(--standout-form-button-text) !important;
+    -webkit-appearance: none;
+    -moz-appearance: none;
+    appearance: none;
 }
 
 .reg-form .help-text {
@@ -276,6 +282,17 @@ label {
     color: var(--standout-form-text);
 }
 
+.reg-form select {
+    background-color: var(--standout-form-bg) !important;
+    color: var(--standout-form-text) !important;
+    border: 1px solid var(--accent-a);
+}
+
+select option {
+    background-color: var(--standout-form-bg);
+    color: var(--standout-form-text);
+}
+
 .reg-form h1,
 .reg-form h2,
 .reg-form h3,
@@ -287,6 +304,49 @@ label {
     margin: 1em 0;
 }
 
+input[type="checkbox"] {
+    -webkit-appearance: none;
+    -moz-appearance: none;
+    appearance: none;
+    width: 16px;
+    height: 16px;
+    border: 1px solid var(--accent-a);
+    border-radius: 3px;
+    background-color: var(--standout-form-bg);
+    cursor: pointer;
+    position: relative;
+    vertical-align: middle;
+    margin-right: 8px;
+}
+
+input[type="checkbox"]:checked {
+    background-color: var(--filter-button-active);
+    border-color: var(--filter-button-active);
+}
+
+input[type="checkbox"]:checked::after {
+    content: "";
+    position: absolute;
+    left: 4px;
+    top: 1px;
+    width: 5px;
+    height: 10px;
+    border: solid var(--standout-form-button-text);
+    border-width: 0 2px 2px 0;
+    transform: rotate(45deg);
+}
+
+/* Specific styles for checkboxes in .reg-form */
+.reg-form input[type="checkbox"] {
+    background-color: var(--standout-form-bg) !important;
+    border-color: var(--accent-a) !important;
+}
+
+.reg-form input[type="checkbox"]:checked {
+    background-color: var(--filter-button-active) !important;
+    border-color: var(--filter-button-active) !important;
+}
+
 .subheading {
     margin-top: -25px;
 }
@@ -330,6 +390,11 @@ input[type="reset"] {
     outline: none;
     font: inherit;
     color: inherit;
+    -webkit-appearance: none;
+    -moz-appearance: none;
+    appearance: none;
+    background-color: var(--filter-button);
+    color: var(--filter-button-text);
 }
 
 hr {
diff --git a/app/jobs/weekly_bookmarks_digest_job.rb b/app/jobs/weekly_bookmarks_digest_job.rb
index ea72a66..76c818f 100644
--- a/app/jobs/weekly_bookmarks_digest_job.rb
+++ b/app/jobs/weekly_bookmarks_digest_job.rb
@@ -1,6 +1,6 @@
 class WeeklyBookmarksDigestJob < ApplicationJob
   queue_as :default
-  retry_on StandardError, attempts: 3, wait: 5.minutes
+  retry_on StandardError, attempts: 2, wait: 5.minutes
 
   def perform(user = nil)
     end_date = Time.current.end_of_day
diff --git a/app/mailers/digest_mailer.rb b/app/mailers/digest_mailer.rb
index b37f889..4df1cfa 100644
--- a/app/mailers/digest_mailer.rb
+++ b/app/mailers/digest_mailer.rb
@@ -6,7 +6,7 @@ class DigestMailer < ApplicationMailer
   #
   def weekly_bookmarks_digest(user, bookmark_ids)
     @user = user
-    @bookmarks = Post.where(id: bookmark_ids).order(created_at: :desc)
+    @bookmarks = Post.where(id: bookmark_ids).order(created_at: :asc)
 
     mail(to: @user.email, subject: 'mind reader :: weekly digest')
   end
diff --git a/app/views/digest_mailer/weekly_bookmarks_digest.html.erb b/app/views/digest_mailer/weekly_bookmarks_digest.html.erb
index 7486925..9d4143a 100644
--- a/app/views/digest_mailer/weekly_bookmarks_digest.html.erb
+++ b/app/views/digest_mailer/weekly_bookmarks_digest.html.erb
@@ -29,12 +29,12 @@
 <body>
   <div class="container">
     <h1 class="header">mind reader</h1>
-    <p class="subheading">Your weekly dispatch and bookmark digest.</p>
+    <p class="subheading">Your weekly dispatch and bookmark digest is below:</p>
 
     <ul>
       <% @bookmarks.group_by { |bookmark| bookmark.created_at.to_date }.each do |date, items| %>
         <li class="date-heading">
-          From <%= date.strftime('%B %e, %Y') %>:
+          <br>From <%= date.strftime('%B %e, %Y') %>:
         </li>
         <% items.each do |item| %>
           <li>
-- 
2.39.5