Make truncate_start a builder method instead

helix-term/src/commands/lsp.rs

@@ -71,7 +71,7 @@ fn sym_picker(
 ) -> FilePicker<lsp::SymbolInformation> {
     // TODO: drop current_path comparison and instead use workspace: bool flag?
     let current_path2 = current_path.clone();
-    let mut picker = FilePicker::new(
+    let picker = FilePicker::new(
         symbols,
         move |symbol| {
             if current_path.as_ref() == Some(&symbol.location.uri) {
@@ -104,8 +104,8 @@ fn sym_picker(
             }
         },
         move |_editor, symbol| Some(location_to_file_location(&symbol.location)),
-    );
-    picker.truncate_start = false;
+    )
+    .truncate_start(false);
     picker
 }
 

helix-term/src/ui/picker.rs

@@ -91,15 +91,25 @@ impl<T> FilePicker<T> {
         callback_fn: impl Fn(&mut Context, &T, Action) + 'static,
         preview_fn: impl Fn(&Editor, &T) -> Option<FileLocation> + 'static,
     ) -> Self {
+        let truncate_start = true;
+        let mut picker = Picker::new(options, format_fn, callback_fn);
+        picker.truncate_start = truncate_start;
+
         Self {
-            picker: Picker::new(options, format_fn, callback_fn),
-            truncate_start: true,
+            picker,
+            truncate_start,
             preview_cache: HashMap::new(),
             read_buffer: Vec::with_capacity(1024),
             file_fn: Box::new(preview_fn),
         }
     }
 
+    pub fn truncate_start(mut self, truncate_start: bool) -> Self {
+        self.truncate_start = truncate_start;
+        self.picker.truncate_start = truncate_start;
+        self
+    }
+
     fn current_file(&self, editor: &Editor) -> Option<FileLocation> {
         self.picker
             .selection()
@@ -176,7 +186,6 @@ impl<T: 'static> Component for FilePicker<T> {
         };
 
         let picker_area = area.with_width(picker_width);
-        self.picker.truncate_start = self.truncate_start;
         self.picker.render(picker_area, surface, cx);
 
         if !render_preview {