Forward move_cursor flag to draw state

The move_cursor flag needs to be set in the wrapped Term draw state, as
that's the one, that is then actually used to draw to the terminal. The
move_cursor flag in the multi draw state doesn't have any effect on
drawing to the terminal.

Author: SuperTux88

src/draw_target.rs

@@ -146,6 +146,15 @@ impl ProgressDrawTarget {
         }
     }
 
+    /// Set whether or not to just move cursor instead of clearing lines
+    pub(crate) fn set_move_cursor(&mut self, move_cursor: bool) {
+        match &mut self.kind {
+            TargetKind::Term { draw_state, .. } => draw_state.move_cursor = move_cursor,
+            TargetKind::TermLike { draw_state, .. } => draw_state.move_cursor = move_cursor,
+            _ => {}
+        }
+    }
+
     /// Apply the given draw state (draws it).
     pub(crate) fn drawable(&mut self, force_draw: bool, now: Instant) -> Option<Drawable<'_>> {
         match &mut self.kind {

src/multi.rs

@@ -58,7 +58,11 @@ impl MultiProgress {
     /// This can reduce flickering, but do not enable it if you intend to change the number of
     /// progress bars.
     pub fn set_move_cursor(&self, move_cursor: bool) {
-        self.state.write().unwrap().move_cursor = move_cursor;
+        self.state
+            .write()
+            .unwrap()
+            .draw_target
+            .set_move_cursor(move_cursor);
     }
 
     /// Set alignment flag
@@ -209,8 +213,6 @@ pub(crate) struct MultiState {
     ordering: Vec<usize>,
     /// Target for draw operation for MultiProgress
     draw_target: ProgressDrawTarget,
-    /// Whether or not to just move cursor instead of clearing lines
-    move_cursor: bool,
     /// Controls how the multi progress is aligned if some of its progress bars get removed, default is `Top`
     alignment: MultiProgressAlignment,
     /// Lines to be drawn above everything else in the MultiProgress. These specifically come from
@@ -227,7 +229,6 @@ impl MultiState {
             free_set: vec![],
             ordering: vec![],
             draw_target,
-            move_cursor: false,
             alignment: MultiProgressAlignment::default(),
             orphan_lines: Vec::new(),
             zombie_lines_count: VisualLines::default(),
@@ -376,10 +377,7 @@ impl MultiState {
         let member = self.members.get_mut(idx).unwrap();
         // alignment is handled by the `MultiProgress`'s underlying draw target, so there is no
         // point in propagating it here.
-        let state = member.draw_state.get_or_insert(DrawState {
-            move_cursor: self.move_cursor,
-            ..Default::default()
-        });
+        let state = member.draw_state.get_or_insert(DrawState::default());
 
         DrawStateWrapper::for_multi(state, &mut self.orphan_lines)
     }

tests/render.rs

@@ -169,6 +169,7 @@ fn multi_progress_single_bar_and_clear() {
     drop(pb1);
     assert_eq!(in_mem.contents(), "");
 }
+
 #[test]
 fn multi_progress_two_bars() {
     let in_mem = InMemoryTerm::new(10, 80);
@@ -417,6 +418,51 @@ Another line printed"#
     );
 }
 
+#[test]
+fn multi_progress_move_cursor() {
+    let in_mem = InMemoryTerm::new(10, 80);
+    let mp =
+        MultiProgress::with_draw_target(ProgressDrawTarget::term_like(Box::new(in_mem.clone())));
+    mp.set_move_cursor(true);
+
+    let pb1 = mp.add(ProgressBar::new(10));
+    pb1.tick();
+    assert_eq!(
+        in_mem.moves_since_last_check(),
+        r#"Str("░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0/10")
+Str("")
+Flush
+"#
+    );
+
+    let pb2 = mp.add(ProgressBar::new(10));
+    pb2.tick();
+    assert_eq!(
+        in_mem.moves_since_last_check(),
+        r#"Up(1)
+Str("░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0/10")
+Str("")
+NewLine
+Str("░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0/10")
+Str("")
+Flush
+"#
+    );
+
+    pb1.inc(1);
+    assert_eq!(
+        in_mem.moves_since_last_check(),
+        r#"Up(2)
+Str("███████░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 1/10")
+Str("")
+NewLine
+Str("░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0/10")
+Str("")
+Flush
+"#
+    );
+}
+
 #[test]
 fn ticker_drop() {
     let in_mem = InMemoryTerm::new(10, 80);