Calendar may have 6 rows (6 weeks)

backend/sql/version_1.sql

@@ -257,7 +257,9 @@ CREATE TABLE [ShoppingEntry] (
     -- In both cases [name], [quantity_value], [quantity_unit] and [Servings] are used to display
     -- the entry instead of [Ingredient] data.
     [ingredient_id] INTEGER,
-    [recipe_scheduled_id] INTEGER, -- Can be null when manually added.
+
+    -- Can be null when manually added (entry is not linked to a recipe).
+    [recipe_scheduled_id] INTEGER,
 
     [is_checked] INTEGER NOT NULL DEFAULT FALSE,
 
@@ -269,16 +271,7 @@ CREATE TABLE [ShoppingEntry] (
 
     CHECK (
         length([name]) <= 255 AND
-        ([is_checked] = TRUE OR [is_checked] = FALSE) AND
-        (
-            [recipe_scheduled_id] IS NULL OR
-            (
-                [name] = '' AND
-                [quantity_value] IS NULL AND
-                [quantity_unit] = '' AND
-                [servings] IS NULL
-            )
-        )
+        ([is_checked] = TRUE OR [is_checked] = FALSE)
     ),
 
     FOREIGN KEY([user_id]) REFERENCES [User]([id]) ON DELETE CASCADE,

backend/templates/calendar.html

@@ -33,7 +33,7 @@
         </div>
     </div>
 
-    <ul class="weekdays">
+    <ul class="days">
         {% let day_names = [
             Sentence::CalendarMondayAbbreviation,
             Sentence::CalendarTuesdayAbbreviation,
@@ -52,12 +52,10 @@
             </li>
         {% endfor %}
 
-    </ul>
-
-    <ul class="days">
-        {% for i in 0..5 %}
+        {# The last row may be hidden if 5 rows is enough to display all days. #}
+        {% for i in 0..6 %}
             {% for j in 0..7 %}
-                {# All days of the current selected month will have the class 'current-month' #}
+                {# All days of the current selected month will have the class 'current-month'. #}
                 <li id="day-grid-{{i}}{{j}}">
                     <div class="number"></div>
                     <div class="scheduled-recipes"></div>
@@ -68,11 +66,7 @@
 
     <div id="hidden-templates-calendar">
         <div class="scheduled-recipe-with-link-and-remove"><a></a>
-            <span class="remove-scheduled-recipe button tooltip">✖
-                <span class="tooltiptext">
-                    {{ context.tr.t(Sentence::CalendarUnschedule) }}
-                </span>
-            </span>
+            <span class="remove-scheduled-recipe button tooltip">✖<span class="tooltiptext">{{ context.tr.t(Sentence::CalendarUnschedule) }}</span></span>
         </div>
 
         <div class="scheduled-recipe"></div>

frontend/src/calendar.rs

@@ -167,22 +167,28 @@ pub fn setup(
     EventListener::new(&days, "click", move |event| {
         let target: Element = event.target().unwrap().dyn_into().unwrap();
 
-        if target.class_name() == "number" && options.can_select_date {
-            let first_day = first_grid_day(
-                state_clone.get_displayed_date(),
-                options.first_day_of_the_week,
-            );
-            let day_grid_id = target.parent_element().unwrap().id();
-            let day_offset = day_grid_id[9..10].parse::<u64>().unwrap() * 7
-                + day_grid_id[10..11].parse::<u64>().unwrap();
-            state_clone.set_selected_date(first_day + Days::new(day_offset));
-            display_month(
-                &calendar_clone,
-                state_clone.clone(),
-                options,
-                recipe_scheduler,
-            );
-        } else if target.class_name().contains("remove-scheduled-recipe") {
+        // FIXME: Replace by an if let + condition when available in stable.
+        match target.selector_upwards::<Element>(".day") {
+            Some(element) if options.can_select_date => {
+                let first_day = first_grid_day(
+                    state_clone.get_displayed_date(),
+                    options.first_day_of_the_week,
+                );
+                let day_grid_id = element.id();
+                let day_offset = day_grid_id[9..10].parse::<u64>().unwrap() * 7
+                    + day_grid_id[10..11].parse::<u64>().unwrap();
+                state_clone.set_selected_date(first_day + Days::new(day_offset));
+                display_month(
+                    &calendar_clone,
+                    state_clone.clone(),
+                    options,
+                    recipe_scheduler,
+                );
+            }
+            _ => (),
+        }
+
+        if target.class_name().contains("remove-scheduled-recipe") {
             spawn_local(async move {
                 // Element id format example: "scheduled-recipe-1-2025-05-15".
                 let element_id = target.parent_element().unwrap().id();
@@ -239,8 +245,6 @@ pub fn setup(
     state
 }
 
-const NB_CALENDAR_ROW: u64 = 5;
-
 fn display_month(
     calendar: &Element,
     state: CalendarState,
@@ -268,34 +272,48 @@ fn display_month(
     let first_day = first_grid_day(date, options.first_day_of_the_week);
     let mut current = first_day;
 
-    for i in 0..NB_CALENDAR_ROW {
+    let nb_of_weeks = {
+        let nb_days_previous_month = if date.month0() == first_day.month0() {
+            0
+        } else {
+            (date.with_day(1).unwrap() - first_day).num_days()
+        };
+        if date.num_days_in_month() + nb_days_previous_month as u8 > 5 * 7 {
+            6
+        } else {
+            5
+        }
+    };
+
+    gloo::console::log!(nb_of_weeks);
+
+    for i in 0..6 {
         for j in 0..7 {
             let day_element: Element = by_id(&format!("day-grid-{}{}", i, j));
-
             let day_content: Element = day_element.selector(".number");
-            day_content.set_inner_html(&current.day().to_string());
 
-            let mut class_name = String::new();
+            if i < nb_of_weeks {
+                day_content.set_inner_html(&current.day().to_string());
 
-            if current.month() == date.month() {
-                if !class_name.is_empty() {
-                    class_name += " ";
+                let mut classes = vec!["day".to_string()];
+
+                if current.month() == date.month() {
+                    classes.push("current-month".to_string());
                 }
-                class_name += "current-month";
-            }
-            if current == Local::now().date_naive() {
-                if !class_name.is_empty() {
-                    class_name += " ";
+                if current == Local::now().date_naive() {
+                    classes.push("today".to_string());
                 }
-                class_name += "today";
-            }
-            if options.can_select_date && current == state.get_selected_date() {
-                if !class_name.is_empty() {
-                    class_name += " ";
+
+                if options.can_select_date && current == state.get_selected_date() {
+                    classes.push("selected-day".to_string());
                 }
-                class_name += "selected-day"
+
+                day_element.set_class_name(&classes.join(" "));
+            } else {
+                gloo::console::log!("ok");
+                day_element.set_class_name("");
+                day_content.set_inner_html("");
             }
-            day_element.set_class_name(&class_name);
 
             current = current + Days::new(1);
         }
@@ -304,7 +322,7 @@ fn display_month(
     // Load and display scheduled recipes.
     spawn_local(async move {
         let scheduled_recipes = recipe_scheduler
-            .get_scheduled_recipes(first_day, first_day + Days::new(NB_CALENDAR_ROW * 7))
+            .get_scheduled_recipes(first_day, first_day + Days::new(nb_of_weeks * 7))
             .await
             .unwrap();
 

frontend/src/utils.rs

@@ -14,6 +14,10 @@ pub trait SelectorExt {
     where
         T: JsCast;
 
+    fn selector_upwards<T>(&self, selector: &str) -> Option<T>
+    where
+        T: JsCast;
+
     fn deep_clone(&self) -> Self;
 }
 
@@ -41,6 +45,25 @@ impl SelectorExt for Element {
             .collect()
     }
 
+    fn selector_upwards<T>(&self, selector: &str) -> Option<T>
+    where
+        T: JsCast,
+    {
+        let mut current = self.clone();
+        // while let Some(parent) = current.parent_element() {
+        loop {
+            if let Ok(true) = current.matches(selector) {
+                return Some(current.dyn_into::<T>().unwrap());
+            }
+
+            if let Some(parent) = current.parent_element() {
+                current = parent;
+            } else {
+                return None;
+            }
+        }
+    }
+
     fn deep_clone(&self) -> Self {
         self.clone_node_with_deep(true)
             .unwrap()