.gitignore

@@ -1,3 +1,2 @@
 /target
 **/*.rs.bk
-tracc_*.json

src/layout.rs

@@ -20,12 +20,12 @@ pub fn selectable_list<'a, C: AsRef<str>>(
 
 pub fn layout(r: Rect) -> Vec<Rect> {
     Layout::default()
-        .direction(Direction::Horizontal)
+        .direction(Direction::Vertical)
         .constraints(
             [
-                Constraint::Percentage(0),
-                Constraint::Percentage(60),
                 Constraint::Percentage(40),
+                Constraint::Percentage(40),
+                Constraint::Percentage(20),
             ]
             .as_ref(),
         )

src/listview.rs

@@ -12,10 +12,6 @@ pub trait ListView<T: fmt::Display + Clone> {
     fn normal_mode(&mut self);
 
     // selection manipulation
-    fn selection_first(&mut self) {
-        *self.selection_pointer() = 0;
-    }
-
     fn selection_up(&mut self) {
         *self.selection_pointer() = self.selection_pointer().saturating_sub(1);
     }
@@ -25,10 +21,6 @@ pub trait ListView<T: fmt::Display + Clone> {
             (*self.selection_pointer() + 1).min(self.list().len().saturating_sub(1));
     }
 
-    fn selection_last(&mut self) {
-        *self.selection_pointer() = self.list().len().saturating_sub(1);
-    }
-
     // adding/removing elements
     fn insert(&mut self, item: T, position: Option<usize>) {
         let pos = position.unwrap_or(*self.selection_pointer());

src/timesheet.rs

@@ -2,7 +2,7 @@ use super::listview::ListView;
 use itertools::Itertools;
 use serde::{Deserialize, Serialize};
 use serde_json::from_reader;
-use std::{collections, default, fmt, fs, io};
+use std::{collections, default, fmt, fs, io, iter};
 use time::{Duration, OffsetDateTime, Time};
 
 pub struct TimeSheet {
@@ -13,13 +13,12 @@ pub struct TimeSheet {
 
 const MAIN_PAUSE_TEXT: &str = "pause";
 const PAUSE_TEXTS: [&str; 4] = [MAIN_PAUSE_TEXT, "lunch", "mittag", "break"];
-const END_TEXT: &str = "end";
 
 lazy_static! {
     static ref OVERRIDE_REGEX: regex::Regex = regex::Regex::new("\\[(.*)\\]").unwrap();
 }
 
-#[derive(Serialize, Deserialize, Clone, Debug, PartialEq, Eq)]
+#[derive(Serialize, Deserialize, Clone, Debug)]
 pub struct TimePoint {
     text: String,
     time: Time,
@@ -29,16 +28,11 @@ impl TimePoint {
     pub fn new(text: &str) -> Self {
         Self {
             text: String::from(text),
-            time: now(),
+            time: OffsetDateTime::now_local().time(),
         }
     }
 }
 
-fn now() -> Time {
-    let raw_time = OffsetDateTime::now_local().time();
-    Time::try_from_hms(raw_time.hour(), raw_time.minute(), 0).unwrap()
-}
-
 impl fmt::Display for TimePoint {
     fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
         write!(f, "[{}] {}", self.time.format("%H:%M"), self.text)
@@ -78,11 +72,9 @@ fn effective_text(s: String) -> String {
 
 impl TimeSheet {
     pub fn open_or_create(path: &str) -> Self {
-        let times = read_times(path).unwrap_or_else(|| vec![TimePoint::new("start")]);
-        let selected = times.len().saturating_sub(1);
         Self {
-            times,
+            times: read_times(path).unwrap_or_else(|| vec![TimePoint::new("start")]),
-            selected,
+            selected: 0,
             register: None,
         }
     }
@@ -98,21 +90,17 @@ impl TimeSheet {
     pub fn shift_current(&mut self, minutes: i64) {
         let time = &mut self.times[self.selected].time;
         *time += Duration::minutes(minutes);
-        *time -= Duration::minutes(time.minute() as i64 % 5);
+        *time -= Duration::minutes(time.minute() as i64 % 5)
-        let timepoint = self.times[self.selected].clone();
-        self.times.sort_by_key(|tp| tp.time);
-        self.selected = self.times.iter().position(|tp| tp == &timepoint).unwrap();
     }
 
     fn current(&self) -> &TimePoint {
         &self.times[self.selected]
     }
 
-    fn grouped_times(&self) -> collections::BTreeMap<String, Duration> {
+    fn grouped_times(&self) -> impl Iterator<Item = (String, Duration)> {
-        let last_time = self.times.last();
         self.times
             .iter()
-            .chain(TimeSheet::maybe_end_time(last_time).iter())
+            .chain(iter::once(&TimePoint::new("end")))
             .tuple_windows()
             .map(|(prev, next)| (prev.text.clone(), next.time - prev.time))
             // Fold into a map to group by description.
@@ -123,45 +111,26 @@ impl TimeSheet {
                     .or_insert_with(Duration::zero) += duration;
                 map
             })
-    }
+            .into_iter()
-
-    fn maybe_end_time(last_time: Option<&TimePoint>) -> Option<TimePoint> {
-        match last_time {
-            Some(tp) if PAUSE_TEXTS.contains(&&tp.text[..]) => None,
-            Some(tp) if tp.text == END_TEXT => None,
-            Some(tp) if tp.time > now() => None,
-            _ => Some(TimePoint::new(END_TEXT)),
-        }
     }
 
     pub fn time_by_tasks(&self) -> String {
         self.grouped_times()
-            .into_iter()
-            .filter(|(text, _)| text != MAIN_PAUSE_TEXT)
             .map(|(text, duration)| format!("{}: {}", text, format_duration(&duration)))
-            .join("\n")
+            .join(" | ")
     }
 
     pub fn sum_as_str(&self) -> String {
-        let total = self.grouped_times()
+        let total = self
-            .into_iter()
+            .grouped_times()
             .filter(|(text, _)| text != MAIN_PAUSE_TEXT)
             .fold(Duration::zero(), |total, (_, d)| total + d);
         format_duration(&total)
     }
-
-    pub fn pause_time(&self) -> String {
-        let times = self.grouped_times();
-        let duration = times
-            .get(MAIN_PAUSE_TEXT)
-            .map(Duration::clone)
-            .unwrap_or_else(Duration::zero);
-        format!("{}: {}", MAIN_PAUSE_TEXT, format_duration(&duration))
-    }
 }
 
 fn format_duration(d: &Duration) -> String {
-    format!("{}:{:02}", d.whole_hours(), d.whole_minutes() % 60)
+    format!("{}:{:02}", d.whole_hours(), d.whole_minutes().max(1) % 60)
 }
 
 impl ListView<TimePoint> for TimeSheet {

src/tracc.rs

@@ -39,7 +39,7 @@ impl Tracc {
             times: TimeSheet::open_or_create(JSON_PATH_TIME),
             terminal,
             input_mode: Mode::Normal,
-            focus: Focus::Bottom,
+            focus: Focus::Top,
         }
     }
 
@@ -51,7 +51,7 @@ impl Tracc {
                     Focus::Bottom => $action(&mut self.times, $($arg,)*),
                 }
             };
-        }
+        };
 
         let mut inputs = io::stdin().keys();
         loop {
@@ -63,13 +63,6 @@ impl Tracc {
                     Key::Char('q') => break,
                     Key::Char('j') => with_focused!(ListView::selection_down),
                     Key::Char('k') => with_focused!(ListView::selection_up),
-                    Key::Char('G') => with_focused!(ListView::selection_first),
-                    // gg
-                    Key::Char('g') => {
-                        if let Some(Ok(Key::Char('g'))) = inputs.next() {
-                            with_focused!(ListView::selection_last);
-                        }
-                    }
                     Key::Char('o') => {
                         with_focused!(ListView::insert, Default::default(), None);
                         self.set_mode(Mode::Insert)?;
@@ -94,6 +87,12 @@ impl Tracc {
                         }
                     }
                     Key::Char('p') => with_focused!(ListView::paste),
+                    Key::Char('\t') => {
+                        self.focus = match self.focus {
+                            Focus::Top => Focus::Bottom,
+                            Focus::Bottom => Focus::Top,
+                        }
+                    }
                     _ => (),
                 },
                 Mode::Insert => match input {
@@ -125,9 +124,8 @@ impl Tracc {
 
     fn refresh(&mut self) -> Result<(), io::Error> {
         let summary_content = [Text::raw(format!(
-            "Sum for today: {}\n{}\n\n{}",
+            "Sum for today: {}\n{}",
             self.times.sum_as_str(),
-            self.times.pause_time(),
             self.times.time_by_tasks()
         ))];
         let mut summary = Paragraph::new(summary_content.iter())