diff --git a/fitnesstrax/app/resources/style.css b/fitnesstrax/app/resources/style.css index 696a917..b7c7dec 100644 --- a/fitnesstrax/app/resources/style.css +++ b/fitnesstrax/app/resources/style.css @@ -3,7 +3,7 @@ } .welcome__title { - font-size: larger; + font-size: x-large; padding: 8px; } @@ -28,21 +28,14 @@ .date-range-picker__date-field { margin: 8px; - font-size: x-large; } .date-range-picker__search-button { margin: 8px; - font-size: x-large; } .date-range-picker__range-button { margin: 8px; - font-size: x-large; -} - -.date-field > label { - font-size: x-large; } .date-field__year { @@ -61,11 +54,14 @@ padding: 8px; } -.day-summary__date { - font-size: larger; +.day-summary > *:not(:last-child) { margin-bottom: 8px; } +.day-summary__date { + font-size: x-large; +} + .day-summary__weight { margin: 4px; } diff --git a/fitnesstrax/app/src/components/day.rs b/fitnesstrax/app/src/components/day.rs index 0402829..f2fa124 100644 --- a/fitnesstrax/app/src/components/day.rs +++ b/fitnesstrax/app/src/components/day.rs @@ -91,17 +91,16 @@ impl DaySummary { if let Some(w) = view_model.weight() { label.set_label(&w.to_string()) } - row.append(&label); - - self.append(&label); let label = gtk::Label::builder() .halign(gtk::Align::Start) - .css_classes(["day-summary__weight"]) + .css_classes(["day-summary__steps"]) .build(); if let Some(s) = view_model.steps() { label.set_label(&format!("{} steps", s)); } + + row.append(&label); row.append(&label); self.append(&row); diff --git a/fitnesstrax/app/src/components/time_distance.rs b/fitnesstrax/app/src/components/time_distance.rs index 6ad5c3a..b06ca8f 100644 --- a/fitnesstrax/app/src/components/time_distance.rs +++ b/fitnesstrax/app/src/components/time_distance.rs @@ -49,7 +49,7 @@ pub fn time_distance_summary( (false, false) => None, }; - text.map(|text| gtk::Label::new(Some(&text))) + text.map(|text| gtk::Label::builder().halign(gtk::Align::Start).label(&text).build()) } pub fn time_distance_detail(record: ft_core::TimeDistance) -> gtk::Box { diff --git a/fitnesstrax/app/src/types.rs b/fitnesstrax/app/src/types.rs index 8ed75fd..36a7252 100644 --- a/fitnesstrax/app/src/types.rs +++ b/fitnesstrax/app/src/types.rs @@ -79,12 +79,12 @@ impl TimeFormatter { match parts.len() { 0 => Err(ParseError), 1 => Err(ParseError), - 2 => Ok(TimeFormatter( - chrono::NaiveTime::from_hms_opt(parts[0], parts[1], 0).unwrap(), - )), - 3 => Ok(TimeFormatter( - chrono::NaiveTime::from_hms_opt(parts[0], parts[1], parts[2]).unwrap(), - )), + 2 => chrono::NaiveTime::from_hms_opt(parts[0], parts[1], 0) + .map(|v| TimeFormatter(v)) + .ok_or(ParseError), + 3 => chrono::NaiveTime::from_hms_opt(parts[0], parts[1], parts[2]) + .map(|v| TimeFormatter(v)) + .ok_or(ParseError), _ => Err(ParseError), } } diff --git a/fitnesstrax/app/src/views/historical_view.rs b/fitnesstrax/app/src/views/historical_view.rs index da025b4..2d04d74 100644 --- a/fitnesstrax/app/src/views/historical_view.rs +++ b/fitnesstrax/app/src/views/historical_view.rs @@ -55,6 +55,7 @@ impl ObjectSubclass for HistoricalViewPrivate { list_view: gtk::ListView::builder() .factory(&factory) .single_click_activate(true) + .show_separators(true) .build(), date_range_picker, };