add comment about move_to_line_end

nushell · Apr 12, 2023 · 7d5d692 · 7d5d692
1 parent 0a46a2e
commit 7d5d692
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/engine.rs b/src/engine.rs
@@ -1023,7 +1023,7 @@ impl Reedline {
                 {
                     Some(Ok(())) => {
                         self.update_buffer_from_history();
-                        // TODO: are these needed/correct?
+                        // Move to end of first line, see `Self::previous_history()`.
                         self.editor.move_to_start(UndoBehavior::HistoryNavigation);
                         self.editor
                             .move_to_line_end(UndoBehavior::HistoryNavigation);
@@ -1092,6 +1092,7 @@ impl Reedline {
             .back(self.history.as_ref())
             .expect("todo: error handling");
         self.update_buffer_from_history();
+        // Move to end of *first* line, so that pressing up again goes directly to previous item.
         self.editor.move_to_start(UndoBehavior::HistoryNavigation);
         self.editor
             .move_to_line_end(UndoBehavior::HistoryNavigation);