From 0f5af82cb5d8d2baaed36f82fc420844d8ed7f2b Mon Sep 17 00:00:00 2001
From: Savanni D'Gerinel <savanni@luminescent-dreams.com>
Date: Fri, 26 Jan 2024 09:53:42 -0500
Subject: [PATCH] Build some convenienc functions for measurement entry fields

Move the weight field into text_entry
---
 fitnesstrax/app/src/components/day.rs         |  1 -
 fitnesstrax/app/src/components/mod.rs         |  2 +-
 fitnesstrax/app/src/components/text_entry.rs  |  1 +
 .../app/src/components/time_distance.rs       | 50 +++++++++++++++++++
 fitnesstrax/app/src/components/weight.rs      |  1 +
 fitnesstrax/app/src/views/historical_view.rs  |  5 +-
 6 files changed, 55 insertions(+), 5 deletions(-)

diff --git a/fitnesstrax/app/src/components/day.rs b/fitnesstrax/app/src/components/day.rs
index 025235a..4be4272 100644
--- a/fitnesstrax/app/src/components/day.rs
+++ b/fitnesstrax/app/src/components/day.rs
@@ -22,7 +22,6 @@ use crate::{
     },
     view_models::DayDetailViewModel,
 };
-use dimensioned::si;
 use ft_core::{RecordType, TraxRecord};
 use glib::Object;
 use gtk::{prelude::*, subclass::prelude::*};
diff --git a/fitnesstrax/app/src/components/mod.rs b/fitnesstrax/app/src/components/mod.rs
index 792db09..59f7347 100644
--- a/fitnesstrax/app/src/components/mod.rs
+++ b/fitnesstrax/app/src/components/mod.rs
@@ -27,7 +27,7 @@ mod steps;
 pub use steps::{steps_editor, Steps};
 
 mod text_entry;
-pub use text_entry::{weight_field, TextEntry};
+pub use text_entry::{distance_field, duration_field, time_field, weight_field, TextEntry};
 
 mod time_distance;
 pub use time_distance::{time_distance_detail, time_distance_summary};
diff --git a/fitnesstrax/app/src/components/text_entry.rs b/fitnesstrax/app/src/components/text_entry.rs
index ff8c206..0cf4ad5 100644
--- a/fitnesstrax/app/src/components/text_entry.rs
+++ b/fitnesstrax/app/src/components/text_entry.rs
@@ -17,6 +17,7 @@ You should have received a copy of the GNU General Public License along with Fit
 use crate::types::{
     DistanceFormatter, DurationFormatter, FormatOption, ParseError, TimeFormatter, WeightFormatter,
 };
+use dimensioned::si;
 use gtk::prelude::*;
 use std::{cell::RefCell, rc::Rc};
 
diff --git a/fitnesstrax/app/src/components/time_distance.rs b/fitnesstrax/app/src/components/time_distance.rs
index 201b284..f7fe9d7 100644
--- a/fitnesstrax/app/src/components/time_distance.rs
+++ b/fitnesstrax/app/src/components/time_distance.rs
@@ -17,6 +17,7 @@ You should have received a copy of the GNU General Public License along with Fit
 // use crate::components::{EditView, ParseError, TextEntry};
 // use chrono::{Local, NaiveDate};
 // use dimensioned::si;
+use crate::components::distance_field;
 use dimensioned::si;
 use ft_core::{RecordType, TimeDistance};
 use glib::Object;
@@ -102,5 +103,54 @@ pub fn time_distance_detail(type_: ft_core::RecordType, record: ft_core::TimeDis
             )
             .build(),
     );
+
     layout
 }
+
+#[derive(Default)]
+pub struct TimeDistanceEditPrivate {
+    type_: RefCell<Option<ft_core::RecordType>>,
+    record: RefCell<Option<ft_core::RecordType>>,
+}
+
+#[glib::object_subclass]
+impl ObjectSubclass for TimeDistanceEditPrivate {
+    const NAME: &'static str = "TimeDistanceEdit";
+    type Type = TimeDistanceEdit;
+    type ParentType = gtk::Box;
+}
+
+impl ObjectImpl for TimeDistanceEditPrivate {}
+impl WidgetImpl for TimeDistanceEditPrivate {}
+impl BoxImpl for TimeDistanceEditPrivate {}
+
+glib::wrapper! {
+    pub struct TimeDistanceEdit(ObjectSubclass<TimeDistanceEditPrivate>) @extends gtk::Box, gtk::Widget, @implements gtk::Orientable;
+}
+
+impl Default for TimeDistanceEdit {
+    fn default() -> Self {
+        let s: Self = Object::builder().build();
+        s.set_orientation(gtk::Orientation::Horizontal);
+        s.set_css_classes(&["time-distance-edit"]);
+
+        s
+    }
+}
+
+impl TimeDistanceEdit {
+    fn empty<OnUpdate>(on_update: OnUpdate) -> Self
+    where
+        OnUpdate: Fn(&ft_core::RecordType, &ft_core::TimeDistance),
+    {
+        let s = Self::default();
+
+        s
+    }
+
+    /*
+    fn with_record<OnUpdate>(type_: ft_core::RecordType, record: ft_core::TimeDistance, on_update: OnUpdate) -> Self
+    where OnUpdate: Fn(&ft_core::RecordType, &ft_core::TimeDistance) {
+    }
+    */
+}
diff --git a/fitnesstrax/app/src/components/weight.rs b/fitnesstrax/app/src/components/weight.rs
index 23ab48f..b3b07a4 100644
--- a/fitnesstrax/app/src/components/weight.rs
+++ b/fitnesstrax/app/src/components/weight.rs
@@ -18,6 +18,7 @@ use crate::{
     components::TextEntry,
     types::{FormatOption, WeightFormatter},
 };
+use dimensioned::si;
 use gtk::prelude::*;
 
 pub struct WeightLabel {
diff --git a/fitnesstrax/app/src/views/historical_view.rs b/fitnesstrax/app/src/views/historical_view.rs
index 2e80e40..b388769 100644
--- a/fitnesstrax/app/src/views/historical_view.rs
+++ b/fitnesstrax/app/src/views/historical_view.rs
@@ -56,6 +56,7 @@ impl ObjectSubclass for HistoricalViewPrivate {
                 .single_click_activate(true)
                 .build(),
         };
+
         factory.connect_bind({
             let app = s.app.clone();
             move |_, list_item| {
@@ -122,9 +123,7 @@ impl HistoricalView {
         s.imp().list_view.connect_activate({
             let on_select_day = on_select_day.clone();
             move |s, idx| {
-                // This gets triggered whenever the user clicks on an item on the list. What we
-                // actually want to do here is to open a modal dialog that shows all of the details of
-                // the day and which allows the user to edit items within that dialog.
+                // This gets triggered whenever the user clicks on an item on the list.
                 let item = s.model().unwrap().item(idx).unwrap();
                 let records = item.downcast_ref::<DayRecords>().unwrap();
                 on_select_day(records.date(), records.records());