show error when failing to navigate to directory

src/editor.rs

@@ -42,9 +42,9 @@ struct Cursor {
 type Line = Range<usize>;
 
 impl Editor {
-	pub fn open_file(clipboard: Clipboard, path: PathBuf) -> Option<Self> {
+	pub fn open_file(clipboard: Clipboard, path: PathBuf) -> std::io::Result<Self> {
-		let text = fs::read_to_string(&path).ok()?;
+		let text = fs::read_to_string(&path)?;
-		Some(Editor {
+		Ok(Editor {
 			text,
 			lines: Vec::new(),
 			scroll: 0,

src/main.rs

@@ -4,7 +4,7 @@ use crossterm::{
 	execute, queue,
 	style::{Color, Colors, ResetColor, SetColors},
 	terminal::{
-		disable_raw_mode, enable_raw_mode, Clear, ClearType, EnterAlternateScreen,
+		self, disable_raw_mode, enable_raw_mode, Clear, ClearType, EnterAlternateScreen,
 		LeaveAlternateScreen,
 	},
 };
@@ -33,6 +33,7 @@ struct Navigator {
 	path: PathBuf,
 	init_path: PathBuf,
 	immediate_open: bool,
+	message: Option<String>,
 }
 
 impl Navigator {
@@ -49,7 +50,7 @@ impl Navigator {
 				path = arg.canonicalize().unwrap();
 				break;
 			} else if arg.is_file() {
-				if let Some(editor) = Editor::open_file(clipboard.clone(), arg) {
+				if let Ok(editor) = Editor::open_file(clipboard.clone(), arg) {
 					editors.push(editor);
 				}
 			} else {
@@ -68,6 +69,7 @@ impl Navigator {
 			init_path: path.clone(),
 			path,
 			immediate_open,
+			message: None,
 		}
 	}
 
@@ -82,6 +84,7 @@ impl Navigator {
 		loop {
 			self.get_files();
 			self.draw();
+			self.message = None;
 			self.input();
 		}
 	}
@@ -123,6 +126,11 @@ impl Navigator {
 			queue!(stdout(), ResetColor).unwrap();
 		}
 
+		if let Some(text) = &self.message {
+			queue!(stdout(), MoveTo(0, terminal::size().unwrap().1)).unwrap();
+			print!("{text}");
+		}
+
 		stdout().flush().unwrap();
 	}
 
@@ -144,6 +152,10 @@ impl Navigator {
 		}
 	}
 
+	fn message(&mut self, text: String) {
+		self.message = Some(text);
+	}
+
 	fn nav_up(&mut self) {
 		self.selected = self.selected.saturating_sub(1);
 	}
@@ -159,39 +171,52 @@ impl Navigator {
 			let i = self.selected - self.editors.len();
 			if i == 0 {
 				if let Some(parent) = self.path.parent() {
-					self.path = parent.to_owned();
+					self.set_path(self.path.join(parent));
-					env::set_current_dir(&self.path).unwrap();
 				}
 			} else {
 				let path = &self.files[i];
 				if path.is_dir() {
-					self.path = self.path.join(path);
+					self.set_path(self.path.join(path));
-					env::set_current_dir(&self.path).unwrap();
-					self.selected = self.editors.len();
 				} else if path.is_file() {
 					let path = path.canonicalize().unwrap();
-					self.selected = self.editors.len();
+					let mut editor_index = self.editors.len();
 					for (i, editor) in self.editors.iter().enumerate() {
 						if editor.path() == Some(&path) {
-							self.selected = i;
+							editor_index = i;
 							break;
 						}
 					}
-					if self.selected == self.editors.len() {
+					if editor_index == self.editors.len() {
-						if let Some(editor) =
+						match Editor::open_file(
-							Editor::open_file(self.clipboard.clone(), path.canonicalize().unwrap())
+							self.clipboard.clone(),
-						{
+							path.canonicalize().unwrap(),
-							self.editors.push(editor);
+						) {
-						} else {
+							Ok(editor) => self.editors.push(editor),
-							return;
+							Err(err) => {
+								self.message(format!("Could not open file: {err}"));
+								return;
+							}
 						}
 					}
+					self.selected = editor_index;
 					self.open_selected();
 				}
 			}
 		}
 	}
 
+	fn set_path(&mut self, new_path: PathBuf) {
+		match env::set_current_dir(&new_path) {
+			Ok(()) => {
+				self.path = new_path;
+				self.selected = self.editors.len();
+			}
+			Err(err) => {
+				self.message(format!("Could not navigate to directory: {err}"));
+			}
+		}
+	}
+
 	fn open_selected(&mut self) {
 		if self.selected < self.editors.len() {
 			self.editors[self.selected].enter();