diff --git a/helix-core/src/syntax.rs b/helix-core/src/syntax.rs index dfd7aec39..c8cb05573 100644 --- a/helix-core/src/syntax.rs +++ b/helix-core/src/syntax.rs @@ -95,7 +95,6 @@ fn load_runtime_file(language: &str, filename: &str) -> Result Result> { - use std::fmt; use std::path::PathBuf; #[derive(rust_embed::RustEmbed)] diff --git a/helix-syntax/src/lib.rs b/helix-syntax/src/lib.rs index b6c0ecf35..b0ec48d82 100644 --- a/helix-syntax/src/lib.rs +++ b/helix-syntax/src/lib.rs @@ -3,15 +3,7 @@ use libloading::{Library, Symbol}; use tree_sitter::Language; fn replace_dashes_with_underscores(name: &str) -> String { - let mut result = String::with_capacity(name.len()); - for c in name.chars() { - if c == '-' { - result.push('_'); - } else { - result.push(c); - } - } - result + name.replace('-', "_") } #[cfg(unix)] const DYLIB_EXTENSION: &str = "so"; diff --git a/helix-term/src/commands.rs b/helix-term/src/commands.rs index 8bde88a2e..57df47a77 100644 --- a/helix-term/src/commands.rs +++ b/helix-term/src/commands.rs @@ -214,6 +214,7 @@ impl Command { file_picker, buffer_picker, symbol_picker, + last_picker, prepend_to_line, append_to_line, open_below, @@ -2127,6 +2128,17 @@ fn symbol_picker(cx: &mut Context) { ) } +fn last_picker(cx: &mut Context) { + // TODO: last picker does not seemed to work well with buffer_picker + cx.callback = Some(Box::new(|compositor: &mut Compositor| { + if let Some(picker) = compositor.last_picker.take() { + compositor.push(picker); + } + // XXX: figure out how to show error when no last picker lifetime + // cx.editor.set_error("no last picker".to_owned()) + })); +} + // I inserts at the first nonwhitespace character of each line with a selection fn prepend_to_line(cx: &mut Context) { goto_first_nonwhitespace(cx); @@ -3837,6 +3849,8 @@ macro_rules! mode_info { mode_info! { /// space mode space_mode, SPACE_MODE, + /// resume last picker + "'" => last_picker, /// file picker "f" => file_picker, /// buffer picker diff --git a/helix-term/src/compositor.rs b/helix-term/src/compositor.rs index 5fcb552a9..c2cfa3a72 100644 --- a/helix-term/src/compositor.rs +++ b/helix-term/src/compositor.rs @@ -74,6 +74,8 @@ type Terminal = tui::terminal::Terminal>; pub struct Compositor { layers: Vec>, terminal: Terminal, + + pub(crate) last_picker: Option>, } impl Compositor { @@ -83,6 +85,7 @@ impl Compositor { Ok(Self { layers: Vec::new(), terminal, + last_picker: None, }) } @@ -103,8 +106,8 @@ impl Compositor { self.layers.push(layer); } - pub fn pop(&mut self) { - self.layers.pop(); + pub fn pop(&mut self) -> Option> { + self.layers.pop() } pub fn handle_event(&mut self, event: Event, cx: &mut Context) -> bool { diff --git a/helix-term/src/ui/menu.rs b/helix-term/src/ui/menu.rs index 8681e5b1c..5a6f6256a 100644 --- a/helix-term/src/ui/menu.rs +++ b/helix-term/src/ui/menu.rs @@ -90,13 +90,13 @@ impl Menu { pub fn move_up(&mut self) { // TODO: wrap around to end - let pos = self.cursor.map_or(0, |i| i.saturating_sub(1)) % self.options.len(); + let pos = self.cursor.map_or(0, |i| i.saturating_sub(1)) % self.matches.len(); self.cursor = Some(pos); self.adjust_scroll(); } pub fn move_down(&mut self) { - let pos = self.cursor.map_or(0, |i| i + 1) % self.options.len(); + let pos = self.cursor.map_or(0, |i| i + 1) % self.matches.len(); self.cursor = Some(pos); self.adjust_scroll(); } @@ -233,16 +233,16 @@ impl Component for Menu { let max_lens = self.options.iter().fold(vec![0; n], |mut acc, option| { let row = option.row(); // maintain max for each column - for (i, cell) in row.cells.iter().enumerate() { + for (acc, cell) in acc.iter_mut().zip(row.cells.iter()) { let width = cell.content.width(); - if width > acc[i] { - acc[i] = width; + if width > *acc { + *acc = width; } } acc }); - let len = (max_lens.iter().sum::()) + n + 1; // +1: reserve some space for scrollbar + let len = max_lens.iter().sum::() + n + 1; // +1: reserve some space for scrollbar let width = len.min(viewport.0 as usize); self.widths = max_lens @@ -250,9 +250,7 @@ impl Component for Menu { .map(|len| Constraint::Length(len as u16)) .collect(); - const MAX: usize = 10; - let height = std::cmp::min(self.options.len(), MAX); - let height = std::cmp::min(height, viewport.1 as usize); + let height = self.options.len().min(10).min(viewport.1 as usize); self.size = (width as u16, height as u16); diff --git a/helix-term/src/ui/picker.rs b/helix-term/src/ui/picker.rs index d7fc9d866..733be2fc6 100644 --- a/helix-term/src/ui/picker.rs +++ b/helix-term/src/ui/picker.rs @@ -159,7 +159,7 @@ impl Component for Picker { let close_fn = EventResult::Consumed(Some(Box::new(|compositor: &mut Compositor| { // remove the layer - compositor.pop(); + compositor.last_picker = compositor.pop(); }))); match key_event { diff --git a/helix-term/src/ui/prompt.rs b/helix-term/src/ui/prompt.rs index a4cb26f7c..2df1e281f 100644 --- a/helix-term/src/ui/prompt.rs +++ b/helix-term/src/ui/prompt.rs @@ -463,7 +463,7 @@ impl Component for Prompt { code: KeyCode::Enter, .. } => { - if self.line.ends_with('/') { + if self.selection.is_some() && self.line.ends_with('/') { self.completion = (self.completion_fn)(&self.line); self.exit_selection(); } else { diff --git a/helix-view/src/document.rs b/helix-view/src/document.rs index 2f2b74798..84de4c433 100644 --- a/helix-view/src/document.rs +++ b/helix-view/src/document.rs @@ -596,6 +596,7 @@ impl Document { let transaction = helix_core::diff::compare_ropes(self.text(), &rope); self.apply(&transaction, view_id); self.append_changes_to_history(view_id); + self.reset_modified(); // Detect indentation style and line ending. self.detect_indent_style(); diff --git a/runtime/themes/dark_plus.toml b/runtime/themes/dark_plus.toml index 82cc62891..e7584e5f7 100644 --- a/runtime/themes/dark_plus.toml +++ b/runtime/themes/dark_plus.toml @@ -46,10 +46,10 @@ "ui.cursor" = { fg = "cursor", modifiers = ["reversed"] } "ui.cursor.primary" = { fg = "cursor", modifiers = ["reversed"] } -"ui.cursor.match" = { fg = "cursor", modifiers = ['underlined'] } +"ui.cursor.match" = { bg = "#3a3d41", modifiers = ["underlined"] } "ui.selection" = { bg = "#3a3d41" } -"ui.selection.primary" = { bg = "#add6ff26" } +"ui.selection.primary" = { bg = "#264f78" } "ui.linenr" = { fg = "#858585" } "ui.linenr.selected" = { fg = "#c6c6c6" } diff --git a/rustfmt.toml b/rustfmt.toml new file mode 100644 index 000000000..e69de29bb