From 9d4c3015632abba2ae6713874907825f6335b71a Mon Sep 17 00:00:00 2001 From: Blaž Hrastnik Date: Tue, 24 Aug 2021 23:43:05 +0900 Subject: Reduce State use a bit further This is a legacy type that should be fully removed. --- helix-core/src/state.rs | 24 ------------------------ helix-core/src/syntax.rs | 17 ++++++++--------- helix-core/src/transaction.rs | 10 +++++----- 3 files changed, 13 insertions(+), 38 deletions(-) diff --git a/helix-core/src/state.rs b/helix-core/src/state.rs index 7e4a7f70..dcc4b11b 100644 --- a/helix-core/src/state.rs +++ b/helix-core/src/state.rs @@ -1,6 +1,5 @@ use crate::{Rope, Selection}; -/// A state represents the current editor state of a single buffer. #[derive(Debug, Clone)] pub struct State { pub doc: Rope, @@ -15,27 +14,4 @@ impl State { selection: Selection::point(0), } } - - // update/transact: - // update(desc) => transaction ? transaction.doc() for applied doc - // transaction.apply(doc) - // doc.transact(fn -> ... end) - - // replaceSelection (transaction that replaces selection) - // changeByRange - // changes - // slice - // - // getters: - // tabSize - // indentUnit - // languageDataAt() - // - // config: - // indentation - // tabSize - // lineUnit - // syntax - // foldable - // changeFilter/transactionFilter } diff --git a/helix-core/src/syntax.rs b/helix-core/src/syntax.rs index 4bceb73b..64b921e6 100644 --- a/helix-core/src/syntax.rs +++ b/helix-core/src/syntax.rs @@ -1828,15 +1828,14 @@ mod test { #[test] fn test_input_edits() { - use crate::State; use tree_sitter::InputEdit; - let state = State::new("hello world!\ntest 123".into()); + let doc = Rope::from("hello world!\ntest 123"); let transaction = Transaction::change( - &state.doc, + &doc, vec![(6, 11, Some("test".into())), (12, 17, None)].into_iter(), ); - let edits = LanguageLayer::generate_edits(state.doc.slice(..), transaction.changes()); + let edits = LanguageLayer::generate_edits(doc.slice(..), transaction.changes()); // transaction.apply(&mut state); assert_eq!( @@ -1862,13 +1861,13 @@ mod test { ); // Testing with the official example from tree-sitter - let mut state = State::new("fn test() {}".into()); + let mut doc = Rope::from("fn test() {}"); let transaction = - Transaction::change(&state.doc, vec![(8, 8, Some("a: u32".into()))].into_iter()); - let edits = LanguageLayer::generate_edits(state.doc.slice(..), transaction.changes()); - transaction.apply(&mut state.doc); + Transaction::change(&doc, vec![(8, 8, Some("a: u32".into()))].into_iter()); + let edits = LanguageLayer::generate_edits(doc.slice(..), transaction.changes()); + transaction.apply(&mut doc); - assert_eq!(state.doc, "fn test(a: u32) {}"); + assert_eq!(doc, "fn test(a: u32) {}"); assert_eq!( edits, &[InputEdit { diff --git a/helix-core/src/transaction.rs b/helix-core/src/transaction.rs index e20e550f..f465992d 100644 --- a/helix-core/src/transaction.rs +++ b/helix-core/src/transaction.rs @@ -689,21 +689,21 @@ mod test { #[test] fn transaction_change() { - let mut state = State::new("hello world!\ntest 123".into()); + let mut doc = Rope::from("hello world!\ntest 123".into()); let transaction = Transaction::change( &state.doc, // (1, 1, None) is a useless 0-width delete vec![(1, 1, None), (6, 11, Some("void".into())), (12, 17, None)].into_iter(), ); - transaction.apply(&mut state.doc); - assert_eq!(state.doc, Rope::from_str("hello void! 123")); + transaction.apply(&mut doc); + assert_eq!(doc, Rope::from_str("hello void! 123")); } #[test] fn changes_iter() { - let state = State::new("hello world!\ntest 123".into()); + let doc = Rope::from("hello world!\ntest 123".into()); let changes = vec![(6, 11, Some("void".into())), (12, 17, None)]; - let transaction = Transaction::change(&state.doc, changes.clone().into_iter()); + let transaction = Transaction::change(&doc, changes.clone().into_iter()); assert_eq!(transaction.changes_iter().collect::>(), changes); } -- cgit v1.2.3-70-g09d2 From 28919898e98541b3305b08d8f05606c060b8ae00 Mon Sep 17 00:00:00 2001 From: Blaž Hrastnik Date: Thu, 26 Aug 2021 09:21:41 +0900 Subject: fix: KeyEvent::char needs to ignore modifiers Fixes #595 --- helix-view/src/input.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/helix-view/src/input.rs b/helix-view/src/input.rs index 28b204bb..1e0ddfe2 100644 --- a/helix-view/src/input.rs +++ b/helix-view/src/input.rs @@ -15,10 +15,10 @@ pub struct KeyEvent { } impl KeyEvent { - /// If a character was pressed (without modifiers), return it. + /// If a character was pressed, return it. pub fn char(&self) -> Option { match self.code { - KeyCode::Char(ch) if self.modifiers.is_empty() => Some(ch), + KeyCode::Char(ch) => Some(ch), _ => None, } } -- cgit v1.2.3-70-g09d2 From 68bf9fdf02a2462c57c86a7318987d385a289184 Mon Sep 17 00:00:00 2001 From: Blaž Hrastnik Date: Thu, 26 Aug 2021 09:26:38 +0900 Subject: Fix tests broken by the State change --- helix-core/src/transaction.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/helix-core/src/transaction.rs b/helix-core/src/transaction.rs index f465992d..d15db000 100644 --- a/helix-core/src/transaction.rs +++ b/helix-core/src/transaction.rs @@ -689,9 +689,9 @@ mod test { #[test] fn transaction_change() { - let mut doc = Rope::from("hello world!\ntest 123".into()); + let mut doc = Rope::from("hello world!\ntest 123"); let transaction = Transaction::change( - &state.doc, + &doc, // (1, 1, None) is a useless 0-width delete vec![(1, 1, None), (6, 11, Some("void".into())), (12, 17, None)].into_iter(), ); @@ -701,7 +701,7 @@ mod test { #[test] fn changes_iter() { - let doc = Rope::from("hello world!\ntest 123".into()); + let doc = Rope::from("hello world!\ntest 123"); let changes = vec![(6, 11, Some("void".into())), (12, 17, None)]; let transaction = Transaction::change(&doc, changes.clone().into_iter()); assert_eq!(transaction.changes_iter().collect::>(), changes); -- cgit v1.2.3-70-g09d2 From 4bafda3995be01b4624469a4ba305e3d019e3f59 Mon Sep 17 00:00:00 2001 From: Ivan Tham Date: Thu, 26 Aug 2021 23:20:37 +0800 Subject: Change vsp to vs (#647) Follow up on #639 to match vim behavior--- helix-term/src/commands.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helix-term/src/commands.rs b/helix-term/src/commands.rs index 9a7b6510..d21bbe42 100644 --- a/helix-term/src/commands.rs +++ b/helix-term/src/commands.rs @@ -2175,7 +2175,7 @@ mod cmd { }, TypableCommand { name: "vsplit", - alias: Some("vsp"), + alias: Some("vs"), doc: "Open the file in a vertical split.", fun: vsplit, completer: Some(completers::filename), -- cgit v1.2.3-70-g09d2 From dc57f8dc89e4d4d58e9af836690c0929a2092439 Mon Sep 17 00:00:00 2001 From: Yusuf Bera Ertan Date: Thu, 26 Aug 2021 18:29:14 +0300 Subject: feat: merge default languages.toml with user provided languages.toml, add a generic TOML value merge function (#654) * feat: merge default languages.toml with user provided languages.toml * refactor: use catch-all to override all other values for merge toml * tests: add a test case for merging languages configs * refactor: change test module name--- helix-core/src/lib.rs | 83 +++++++++++++++++++++++++++++++++++++++++++ helix-term/src/application.rs | 21 +++++++---- 2 files changed, 97 insertions(+), 7 deletions(-) diff --git a/helix-core/src/lib.rs b/helix-core/src/lib.rs index be01c302..d971464a 100644 --- a/helix-core/src/lib.rs +++ b/helix-core/src/lib.rs @@ -98,6 +98,89 @@ pub fn cache_dir() -> std::path::PathBuf { path } +// right overrides left +pub fn merge_toml_values(left: toml::Value, right: toml::Value) -> toml::Value { + use toml::Value; + + fn get_name(v: &Value) -> Option<&str> { + v.get("name").and_then(Value::as_str) + } + + match (left, right) { + (Value::Array(mut left_items), Value::Array(right_items)) => { + left_items.reserve(right_items.len()); + for rvalue in right_items { + let lvalue = get_name(&rvalue) + .and_then(|rname| left_items.iter().position(|v| get_name(v) == Some(rname))) + .map(|lpos| left_items.remove(lpos)); + let mvalue = match lvalue { + Some(lvalue) => merge_toml_values(lvalue, rvalue), + None => rvalue, + }; + left_items.push(mvalue); + } + Value::Array(left_items) + } + (Value::Table(mut left_map), Value::Table(right_map)) => { + for (rname, rvalue) in right_map { + match left_map.remove(&rname) { + Some(lvalue) => { + let merged_value = merge_toml_values(lvalue, rvalue); + left_map.insert(rname, merged_value); + } + None => { + left_map.insert(rname, rvalue); + } + } + } + Value::Table(left_map) + } + // Catch everything else we didn't handle, and use the right value + (_, value) => value, + } +} + +#[cfg(test)] +mod merge_toml_tests { + use super::merge_toml_values; + + #[test] + fn language_tomls() { + use toml::Value; + + const USER: &str = " + [[language]] + name = \"nix\" + test = \"bbb\" + indent = { tab-width = 4, unit = \" \", test = \"aaa\" } + "; + + let base: Value = toml::from_slice(include_bytes!("../../languages.toml")) + .expect("Couldn't parse built-in langauges config"); + let user: Value = toml::from_str(USER).unwrap(); + + let merged = merge_toml_values(base, user); + let languages = merged.get("language").unwrap().as_array().unwrap(); + let nix = languages + .iter() + .find(|v| v.get("name").unwrap().as_str().unwrap() == "nix") + .unwrap(); + let nix_indent = nix.get("indent").unwrap(); + + // We changed tab-width and unit in indent so check them if they are the new values + assert_eq!( + nix_indent.get("tab-width").unwrap().as_integer().unwrap(), + 4 + ); + assert_eq!(nix_indent.get("unit").unwrap().as_str().unwrap(), " "); + // We added a new keys, so check them + assert_eq!(nix.get("test").unwrap().as_str().unwrap(), "bbb"); + assert_eq!(nix_indent.get("test").unwrap().as_str().unwrap(), "aaa"); + // We didn't change comment-token so it should be same + assert_eq!(nix.get("comment-token").unwrap().as_str().unwrap(), "#"); + } +} + pub use etcetera::home_dir; use etcetera::base_strategy::{choose_base_strategy, BaseStrategy}; diff --git a/helix-term/src/application.rs b/helix-term/src/application.rs index 3d59c33a..1fcca681 100644 --- a/helix-term/src/application.rs +++ b/helix-term/src/application.rs @@ -1,4 +1,4 @@ -use helix_core::syntax; +use helix_core::{merge_toml_values, syntax}; use helix_lsp::{lsp, util::lsp_pos_to_pos, LspProgressMap}; use helix_view::{theme, Editor}; @@ -59,11 +59,16 @@ impl Application { let theme_loader = std::sync::Arc::new(theme::Loader::new(&conf_dir, &helix_core::runtime_dir())); - // load $HOME/.config/helix/languages.toml, fallback to default config - let lang_conf = std::fs::read(conf_dir.join("languages.toml")); - let lang_conf = lang_conf - .as_deref() - .unwrap_or(include_bytes!("../../languages.toml")); + // load default and user config, and merge both + let def_lang_conf: toml::Value = toml::from_slice(include_bytes!("../../languages.toml")) + .expect("Could not parse built-in languages.toml, something must be very wrong"); + let user_lang_conf: Option = std::fs::read(conf_dir.join("languages.toml")) + .ok() + .map(|raw| toml::from_slice(&raw).expect("Could not parse user languages.toml")); + let lang_conf = match user_lang_conf { + Some(value) => merge_toml_values(def_lang_conf, value), + None => def_lang_conf, + }; let theme = if let Some(theme) = &config.theme { match theme_loader.load(theme) { @@ -77,7 +82,9 @@ impl Application { theme_loader.default() }; - let syn_loader_conf = toml::from_slice(lang_conf).expect("Could not parse languages.toml"); + let syn_loader_conf: helix_core::syntax::Configuration = lang_conf + .try_into() + .expect("Could not parse merged (built-in + user) languages.toml"); let syn_loader = std::sync::Arc::new(syntax::Loader::new(syn_loader_conf)); let mut editor = Editor::new( -- cgit v1.2.3-70-g09d2 From 6192f2fa250e3aa10e514547c69ae901aebed657 Mon Sep 17 00:00:00 2001 From: Stuart Hinson Date: Thu, 26 Aug 2021 08:30:47 -0700 Subject: Show hidden files in filename completer (#648) also removes unnecessary clone--- helix-term/src/ui/mod.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/helix-term/src/ui/mod.rs b/helix-term/src/ui/mod.rs index e4871312..f3f8670e 100644 --- a/helix-term/src/ui/mod.rs +++ b/helix-term/src/ui/mod.rs @@ -228,7 +228,8 @@ pub mod completers { let end = input.len()..; - let mut files: Vec<_> = WalkBuilder::new(dir.clone()) + let mut files: Vec<_> = WalkBuilder::new(&dir) + .hidden(false) .max_depth(Some(1)) .build() .filter_map(|file| { -- cgit v1.2.3-70-g09d2 From cec5d437d892767ccbc49cfcb20b4561ce24576d Mon Sep 17 00:00:00 2001 From: Grzegorz Baranski Date: Thu, 26 Aug 2021 19:48:33 +0200 Subject: fix: show current line number even if relative line is on (#656) --- helix-term/src/ui/editor.rs | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/helix-term/src/ui/editor.rs b/helix-term/src/ui/editor.rs index 4da8bfd5..72b8adc1 100644 --- a/helix-term/src/ui/editor.rs +++ b/helix-term/src/ui/editor.rs @@ -462,7 +462,13 @@ impl EditorView { } else { let line = match config.line_number { LineNumber::Absolute => line + 1, - LineNumber::Relative => abs_diff(current_line, line), + LineNumber::Relative => { + if current_line == line { + line + 1 + } else { + abs_diff(current_line, line) + } + } }; format!("{:>5}", line) }; -- cgit v1.2.3-70-g09d2 From fa4caf7e3d43a40073f6966d37f529d5197bd840 Mon Sep 17 00:00:00 2001 From: Brian Shu Date: Thu, 26 Aug 2021 14:45:23 -0400 Subject: remove unsafe --- helix-core/src/transaction.rs | 2 +- helix-view/src/document.rs | 17 +++++------------ 2 files changed, 6 insertions(+), 13 deletions(-) diff --git a/helix-core/src/transaction.rs b/helix-core/src/transaction.rs index d15db000..d682f058 100644 --- a/helix-core/src/transaction.rs +++ b/helix-core/src/transaction.rs @@ -125,7 +125,7 @@ impl ChangeSet { /// In other words, If `this` goes `docA` → `docB` and `other` represents `docB` → `docC`, the /// returned value will represent the change `docA` → `docC`. pub fn compose(self, other: Self) -> Self { - debug_assert!(self.len_after == other.len); + assert!(self.len_after == other.len); // composing fails in weird ways if one of the sets is empty // a: [] len: 0 len_after: 1 | b: [Insert(Tendril(inline: "\n")), Retain(1)] len 1 diff --git a/helix-view/src/document.rs b/helix-view/src/document.rs index a238644a..e890a336 100644 --- a/helix-view/src/document.rs +++ b/helix-view/src/document.rs @@ -102,7 +102,7 @@ pub struct Document { language_server: Option>, } -use std::fmt; +use std::{fmt, mem}; impl fmt::Debug for Document { fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { f.debug_struct("Document") @@ -301,20 +301,13 @@ pub async fn to_writer<'a, W: tokio::io::AsyncWriteExt + Unpin + ?Sized>( Ok(()) } -/// Like std::mem::replace() except it allows the replacement value to be mapped from the -/// original value. -fn take_with(mut_ref: &mut T, closure: F) +fn take_with(mut_ref: &mut T, f: F) where + T: Default, F: FnOnce(T) -> T, { - use std::{panic, ptr}; - - unsafe { - let old_t = ptr::read(mut_ref); - let new_t = panic::catch_unwind(panic::AssertUnwindSafe(|| closure(old_t))) - .unwrap_or_else(|_| ::std::process::abort()); - ptr::write(mut_ref, new_t); - } + let t = mem::take(mut_ref); + let _ = mem::replace(mut_ref, f(t)); } use helix_lsp::lsp; -- cgit v1.2.3-70-g09d2 From bfce4d4f29c12bfd9d6bcb01da976d33beff990e Mon Sep 17 00:00:00 2001 From: Omnikar Date: Thu, 26 Aug 2021 21:03:49 -0400 Subject: Make `v` in select mode switch back to normal mode (#660) * Make `v` in select mode switch back to normal mode * Move select mode toggle to keymap instead of command--- helix-term/src/keymap.rs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/helix-term/src/keymap.rs b/helix-term/src/keymap.rs index 57bcb321..492dc292 100644 --- a/helix-term/src/keymap.rs +++ b/helix-term/src/keymap.rs @@ -529,6 +529,8 @@ impl Default for Keymaps { "home" => goto_line_start, "end" => goto_line_end, "esc" => exit_select_mode, + + "v" => normal_mode, })); let insert = keymap!({ "Insert mode" "esc" => normal_mode, -- cgit v1.2.3-70-g09d2 From 048a3905684b7f27dae20a0eeee37e6583862448 Mon Sep 17 00:00:00 2001 From: Omnikar Date: Thu, 26 Aug 2021 22:45:18 -0400 Subject: Add `Command` column to keymap documentation (#662) --- book/src/keymap.md | 292 ++++++++++++++++++++++++++--------------------------- 1 file changed, 146 insertions(+), 146 deletions(-) diff --git a/book/src/keymap.md b/book/src/keymap.md index 4eb85636..0308da82 100644 --- a/book/src/keymap.md +++ b/book/src/keymap.md @@ -6,121 +6,121 @@ > NOTE: `f`, `F`, `t` and `T` are not confined to the current line. -| Key | Description | -| ----- | ----------- | -| `h`, `Left` | Move left | -| `j`, `Down` | Move down | -| `k`, `Up` | Move up | -| `l`, `Right` | Move right | -| `w` | Move next word start | -| `b` | Move previous word start | -| `e` | Move next word end | -| `W` | Move next WORD start | -| `B` | Move previous WORD start | -| `E` | Move next WORD end | -| `t` | Find 'till next char | -| `f` | Find next char | -| `T` | Find 'till previous char | -| `F` | Find previous char | -| `Home` | Move to the start of the line | -| `End` | Move to the end of the line | -| `PageUp` | Move page up | -| `PageDown` | Move page down | -| `Ctrl-u` | Move half page up | -| `Ctrl-d` | Move half page down | -| `Ctrl-i` | Jump forward on the jumplist TODO: conflicts tab | -| `Ctrl-o` | Jump backward on the jumplist | -| `v` | Enter [select (extend) mode](#select--extend-mode) | -| `g` | Enter [goto mode](#goto-mode) | -| `m` | Enter [match mode](#match-mode) | -| `:` | Enter command mode | -| `z` | Enter [view mode](#view-mode) | -| `Ctrl-w` | Enter [window mode](#window-mode) (maybe will be remove for spc w w later) | -| `Space` | Enter [space mode](#space-mode) | -| `K` | Show documentation for the item under the cursor | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `h`, `Left` | Move left | `move_char_left` | +| `j`, `Down` | Move down | `move_char_right` | +| `k`, `Up` | Move up | `move_line_up` | +| `l`, `Right` | Move right | `move_line_down` | +| `w` | Move next word start | `move_next_word_start` | +| `b` | Move previous word start | `move_prev_word_start` | +| `e` | Move next word end | `move_next_word_end` | +| `W` | Move next WORD start | `move_next_long_word_start` | +| `B` | Move previous WORD start | `move_prev_long_word_start` | +| `E` | Move next WORD end | `move_next_long_word_end` | +| `t` | Find 'till next char | `find_till_char` | +| `f` | Find next char | `find_next_char` | +| `T` | Find 'till previous char | `till_prev_char` | +| `F` | Find previous char | `find_prev_char` | +| `Home` | Move to the start of the line | `goto_line_start` | +| `End` | Move to the end of the line | `goto_line_end` | +| `PageUp` | Move page up | `page_up` | +| `PageDown` | Move page down | `page_down` | +| `Ctrl-u` | Move half page up | `half_page_up` | +| `Ctrl-d` | Move half page down | `half_page_down` | +| `Ctrl-i` | Jump forward on the jumplist TODO: conflicts tab | `jump_forward` | +| `Ctrl-o` | Jump backward on the jumplist | `jump_backward` | +| `v` | Enter [select (extend) mode](#select--extend-mode) | `select_mode` | +| `g` | Enter [goto mode](#goto-mode) | N/A | +| `m` | Enter [match mode](#match-mode) | N/A | +| `:` | Enter command mode | `command_mode` | +| `z` | Enter [view mode](#view-mode) | N/A | +| `Ctrl-w` | Enter [window mode](#window-mode) (maybe will be remove for spc w w later) | N/A | +| `Space` | Enter [space mode](#space-mode) | N/A | +| `K` | Show documentation for the item under the cursor | `hover` | ### Changes -| Key | Description | -| ----- | ----------- | -| `r` | Replace with a character | -| `R` | Replace with yanked text | -| `~` | Switch case of the selected text | -| `` ` `` | Set the selected text to lower case | -| `` Alt-` `` | Set the selected text to upper case | -| `i` | Insert before selection | -| `a` | Insert after selection (append) | -| `I` | Insert at the start of the line | -| `A` | Insert at the end of the line | -| `o` | Open new line below selection | -| `O` | Open new line above selection | -| `u` | Undo change | -| `U` | Redo change | -| `y` | Yank selection | -| `p` | Paste after selection | -| `P` | Paste before selection | -| `"` `` | Select a register to yank to or paste from | -| `>` | Indent selection | -| `<` | Unindent selection | -| `=` | Format selection | -| `d` | Delete selection | -| `c` | Change selection (delete and enter insert mode) | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `r` | Replace with a character | `replace` | +| `R` | Replace with yanked text | `replace_with_yanked` | +| `~` | Switch case of the selected text | `switch_case` | +| `` ` `` | Set the selected text to lower case | `switch_to_lowercase` | +| `` Alt-` `` | Set the selected text to upper case | `switch_to_uppercase` | +| `i` | Insert before selection | `insert_mode` | +| `a` | Insert after selection (append) | `append_mode` | +| `I` | Insert at the start of the line | `prepend_to_line` | +| `A` | Insert at the end of the line | `append_to_line` | +| `o` | Open new line below selection | `open_below` | +| `O` | Open new line above selection | `open_above` | +| `u` | Undo change | `undo` | +| `U` | Redo change | `redo` | +| `y` | Yank selection | `yank` | +| `p` | Paste after selection | `paste_after` | +| `P` | Paste before selection | `paste_before` | +| `"` `` | Select a register to yank to or paste from | `select_register` | +| `>` | Indent selection | `indent` | +| `<` | Unindent selection | `unindent` | +| `=` | Format selection | `format_selections` | +| `d` | Delete selection | `delete_selection` | +| `c` | Change selection (delete and enter insert mode) | `change_selection` | ### Selection manipulation -| Key | Description | -| ----- | ----------- | -| `s` | Select all regex matches inside selections | -| `S` | Split selection into subselections on regex matches | -| `Alt-s` | Split selection on newlines | -| `;` | Collapse selection onto a single cursor | -| `Alt-;` | Flip selection cursor and anchor | -| `C` | Copy selection onto the next line | -| `Alt-C` | Copy selection onto the previous line | -| `(` | Rotate main selection forward | -| `)` | Rotate main selection backward | -| `Alt-(` | Rotate selection contents forward | -| `Alt-)` | Rotate selection contents backward | -| `%` | Select entire file | -| `x` | Select current line, if already selected, extend to next line | -| `X` | Extend selection to line bounds (line-wise selection) | -| | Expand selection to parent syntax node TODO: pick a key | -| `J` | Join lines inside selection | -| `K` | Keep selections matching the regex TODO: overlapped by hover help | -| `Space` | Keep only the primary selection TODO: overlapped by space mode | -| `Ctrl-c` | Comment/uncomment the selections | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `s` | Select all regex matches inside selections | `select_regex` | +| `S` | Split selection into subselections on regex matches | `split_selection` | +| `Alt-s` | Split selection on newlines | `split_selection_on_newline` | +| `;` | Collapse selection onto a single cursor | `collapse_selection` | +| `Alt-;` | Flip selection cursor and anchor | `flip_selections` | +| `C` | Copy selection onto the next line | `copy_selection_on_next_line` | +| `Alt-C` | Copy selection onto the previous line | `copy_selection_on_prev_line` | +| `(` | Rotate main selection forward | `rotate_selections_backward` | +| `)` | Rotate main selection backward | `rotate_selections_forward` | +| `Alt-(` | Rotate selection contents forward | `rotate_selection_contents_backward` | +| `Alt-)` | Rotate selection contents backward | `rotate_selection_contents_forward` | +| `%` | Select entire file | `select_all` | +| `x` | Select current line, if already selected, extend to next line | `extend_line` | +| `X` | Extend selection to line bounds (line-wise selection) | `extend_to_line_bounds` | +| | Expand selection to parent syntax node TODO: pick a key | `expand_selection` | +| `J` | Join lines inside selection | `join_selections` | +| `K` | Keep selections matching the regex TODO: overlapped by hover help | `keep_selections` | +| `Space` | Keep only the primary selection TODO: overlapped by space mode | `keep_primary_selection` | +| `Ctrl-c` | Comment/uncomment the selections | `toggle_comments` | ### Insert Mode -| Key | Description | -| ----- | ----------- | -| `Escape` | Switch to normal mode | -| `Ctrl-x` | Autocomplete | -| `Ctrl-w` | Delete previous word | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `Escape` | Switch to normal mode | `normal_mode` | +| `Ctrl-x` | Autocomplete | `completion` | +| `Ctrl-w` | Delete previous word | `delete_word_backward` | ### Search > TODO: The search implementation isn't ideal yet -- we don't support searching in reverse, or searching via smartcase. -| Key | Description | -| ----- | ----------- | -| `/` | Search for regex pattern | -| `n` | Select next search match | -| `N` | Add next search match to selection | -| `*` | Use current selection as the search pattern | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `/` | Search for regex pattern | `search` | +| `n` | Select next search match | `search_next` | +| `N` | Add next search match to selection | `extend_search_next` | +| `*` | Use current selection as the search pattern | `search_selection` | ### Diagnostics > NOTE: `[` and `]` will likely contain more pair mappings in the style of > [vim-unimpaired](https://github.com/tpope/vim-unimpaired) -| Key | Description | -| ----- | ----------- | -| `[d` | Go to previous diagnostic | -| `]d` | Go to next diagnostic | -| `[D` | Go to first diagnostic in document | -| `]D` | Go to last diagnostic in document | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `[d` | Go to previous diagnostic | `goto_prev_diag` | +| `]d` | Go to next diagnostic | `goto_next_diag` | +| `[D` | Go to first diagnostic in document | `goto_first_diag` | +| `]D` | Go to last diagnostic in document | `goto_last_diag` | ## Select / extend mode @@ -135,14 +135,14 @@ commands to extend the existing selection instead of replacing it. View mode is intended for scrolling and manipulating the view without changing the selection. -| Key | Description | -| ----- | ----------- | -| `z` , `c` | Vertically center the line | -| `t` | Align the line to the top of the screen | -| `b` | Align the line to the bottom of the screen | -| `m` | Align the line to the middle of the screen (horizontally) | -| `j` | Scroll the view downwards | -| `k` | Scroll the view upwards | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `z` , `c` | Vertically center the line | `align_view_center` | +| `t` | Align the line to the top of the screen | `align_view_top` | +| `b` | Align the line to the bottom of the screen | `align_view_bottom` | +| `m` | Align the line to the middle of the screen (horizontally) | `align_view_middle` | +| `j` | Scroll the view downwards | `scroll_down` | +| `k` | Scroll the view upwards | `scroll_up` | ## Goto mode @@ -150,21 +150,21 @@ Jumps to various locations. > NOTE: Some of these features are only available with the LSP present. -| Key | Description | -| ----- | ----------- | -| `g` | Go to the start of the file | -| `e` | Go to the end of the file | -| `h` | Go to the start of the line | -| `l` | Go to the end of the line | -| `s` | Go to first non-whitespace character of the line | -| `t` | Go to the top of the screen | -| `m` | Go to the middle of the screen | -| `b` | Go to the bottom of the screen | -| `d` | Go to definition | -| `y` | Go to type definition | -| `r` | Go to references | -| `i` | Go to implementation | -| `a` | Go to the last accessed/alternate file | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `g` | Go to the start of the file | `goto_file_start` | +| `e` | Go to the end of the file | `goto_last_line` | +| `h` | Go to the start of the line | `goto_line_start` | +| `l` | Go to the end of the line | `goto_line_end` | +| `s` | Go to first non-whitespace character of the line | `goto_first_nonwhitespace` | +| `t` | Go to the top of the screen | `goto_window_top` | +| `m` | Go to the middle of the screen | `goto_window_middle` | +| `b` | Go to the bottom of the screen | `goto_window_bottom` | +| `d` | Go to definition | `goto_definition` | +| `y` | Go to type definition | `goto_type_definition` | +| `r` | Go to references | `goto_reference` | +| `i` | Go to implementation | `goto_implementation` | +| `a` | Go to the last accessed/alternate file | `goto_last_accessed_file` | ## Match mode @@ -172,14 +172,14 @@ Enter this mode using `m` from normal mode. See the relavant section in [Usage](./usage.md) for an explanation about [surround](./usage.md#surround) and [textobject](./usage.md#textobject) usage. -| Key | Description | -| ----- | ----------- | -| `m` | Goto matching bracket | -| `s` `` | Surround current selection with `` | -| `r` `` | Replace surround character `` with `` | -| `d` `` | Delete surround character `` | -| `a` `` | Select around textobject | -| `i` `` | Select inside textobject | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `m` | Goto matching bracket | `match_brackets` | +| `s` `` | Surround current selection with `` | `surround_add` | +| `r` `` | Replace surround character `` with `` | `surround_replace` | +| `d` `` | Delete surround character `` | `surround_delete` | +| `a` `` | Select around textobject | `select_textobject_around` | +| `i` `` | Select inside textobject | `select_textobject_inner` | ## Object mode @@ -189,35 +189,35 @@ TODO: Mappings for selecting syntax nodes (a superset of `[`). This layer is similar to vim keybindings as kakoune does not support window. -| Key | Description | -| ----- | ------------- | -| `w`, `Ctrl-w` | Switch to next window | -| `v`, `Ctrl-v` | Vertical right split | -| `h`, `Ctrl-h` | Horizontal bottom split | -| `q`, `Ctrl-q` | Close current window | +| Key | Description | Command | +| ----- | ------------- | ------- | +| `w`, `Ctrl-w` | Switch to next window | `rotate_view` | +| `v`, `Ctrl-v` | Vertical right split | `vsplit` | +| `h`, `Ctrl-h` | Horizontal bottom split | `hsplit | +| `q`, `Ctrl-q` | Close current window | `wclose` | ## Space mode This layer is a kludge of mappings I had under leader key in neovim. -| Key | Description | -| ----- | ----------- | -| `f` | Open file picker | -| `b` | Open buffer picker | -| `s` | Open symbol picker (current document) | -| `a` | Apply code action | -| `'` | Open last fuzzy picker | -| `w` | Enter [window mode](#window-mode) | -| `space` | Keep primary selection TODO: it's here because space mode replaced it | -| `p` | Paste system clipboard after selections | -| `P` | Paste system clipboard before selections | -| `y` | Join and yank selections to clipboard | -| `Y` | Yank main selection to clipboard | -| `R` | Replace selections by clipboard contents | +| Key | Description | Command | +| ----- | ----------- | ------- | +| `f` | Open file picker | `file_picker` | +| `b` | Open buffer picker | `buffer_picker` | +| `s` | Open symbol picker (current document) | `symbol_picker` | +| `a` | Apply code action | `code_action` | +| `'` | Open last fuzzy picker | `last_picker` | +| `w` | Enter [window mode](#window-mode) | N/A | +| `space` | Keep primary selection TODO: it's here because space mode replaced it | `keep_primary_selection` | +| `p` | Paste system clipboard after selections | `paste_clipboard_after` | +| `P` | Paste system clipboard before selections | `paste_clipboard_before` | +| `y` | Join and yank selections to clipboard | `yank_joined_to_clipboard` | +| `Y` | Yank main selection to clipboard | `yank_main_selection_to_clipboard` | +| `R` | Replace selections by clipboard contents | `replace_selections_with_clipboard` | # Picker -Keys to use within picker. +Keys to use within picker. Remapping currently not supported. | Key | Description | | ----- | ------------- | -- cgit v1.2.3-70-g09d2 From 46f537d4ce0ae9c8f0be8d89f97c3fbe68afabd5 Mon Sep 17 00:00:00 2001 From: Omnikar Date: Fri, 27 Aug 2021 00:09:51 -0400 Subject: Fix missing backtick in `keymap.md` --- book/src/keymap.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/book/src/keymap.md b/book/src/keymap.md index 0308da82..61378863 100644 --- a/book/src/keymap.md +++ b/book/src/keymap.md @@ -193,7 +193,7 @@ This layer is similar to vim keybindings as kakoune does not support window. | ----- | ------------- | ------- | | `w`, `Ctrl-w` | Switch to next window | `rotate_view` | | `v`, `Ctrl-v` | Vertical right split | `vsplit` | -| `h`, `Ctrl-h` | Horizontal bottom split | `hsplit | +| `h`, `Ctrl-h` | Horizontal bottom split | `hsplit` | | `q`, `Ctrl-q` | Close current window | `wclose` | ## Space mode -- cgit v1.2.3-70-g09d2 From 5cee3b634d3763dfdb0312d1dc58f6ebcfccb4bf Mon Sep 17 00:00:00 2001 From: Blaž Hrastnik Date: Fri, 27 Aug 2021 16:39:21 +0900 Subject: ui: prompt: Fix typing with alt --- helix-term/src/ui/prompt.rs | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/helix-term/src/ui/prompt.rs b/helix-term/src/ui/prompt.rs index 19986b5c..7197adea 100644 --- a/helix-term/src/ui/prompt.rs +++ b/helix-term/src/ui/prompt.rs @@ -400,18 +400,6 @@ impl Component for Prompt { }))); match event { - // char or shift char - KeyEvent { - code: KeyCode::Char(c), - modifiers: KeyModifiers::NONE, - } - | KeyEvent { - code: KeyCode::Char(c), - modifiers: KeyModifiers::SHIFT, - } => { - self.insert_char(c); - (self.callback_fn)(cx, &self.line, PromptEvent::Update); - } KeyEvent { code: KeyCode::Char('c'), modifiers: KeyModifiers::CONTROL, @@ -539,6 +527,14 @@ impl Component for Prompt { code: KeyCode::Char('q'), modifiers: KeyModifiers::CONTROL, } => self.exit_selection(), + // any char event that's not combined with control or mapped to any other combo + KeyEvent { + code: KeyCode::Char(c), + modifiers, + } if !modifiers.contains(KeyModifiers::CONTROL) => { + self.insert_char(c); + (self.callback_fn)(cx, &self.line, PromptEvent::Update); + } _ => (), }; -- cgit v1.2.3-70-g09d2 From f22e0aa2ae4169851b43d2225e6a43ef36442659 Mon Sep 17 00:00:00 2001 From: voroskoi Date: Sat, 28 Aug 2021 06:32:01 +0200 Subject: Add zig tree-sitter support (#631) * Add initial zig tree-sitter support * zig/highlights.scm: remove unnecessary queries * Add zig/indents.toml--- .gitmodules | 4 + helix-syntax/languages/tree-sitter-zig | 1 + languages.toml | 12 ++ runtime/queries/zig/highlights.scm | 198 +++++++++++++++++++++++++++++++++ runtime/queries/zig/indents.toml | 12 ++ 5 files changed, 227 insertions(+) create mode 160000 helix-syntax/languages/tree-sitter-zig create mode 100644 runtime/queries/zig/highlights.scm create mode 100644 runtime/queries/zig/indents.toml diff --git a/.gitmodules b/.gitmodules index e750198a..0e015658 100644 --- a/.gitmodules +++ b/.gitmodules @@ -102,3 +102,7 @@ path = helix-syntax/languages/tree-sitter-protobuf url = https://github.com/yusdacra/tree-sitter-protobuf.git shallow = true +[submodule "helix-syntax/languages/tree-sitter-zig"] + path = helix-syntax/languages/tree-sitter-zig + url = https://github.com/maxxnino/tree-sitter-zig + shallow = true diff --git a/helix-syntax/languages/tree-sitter-zig b/helix-syntax/languages/tree-sitter-zig new file mode 160000 index 00000000..049162be --- /dev/null +++ b/helix-syntax/languages/tree-sitter-zig @@ -0,0 +1 @@ +Subproject commit 049162bea8a44e1a4acd01b06e1c8672d9231a86 diff --git a/languages.toml b/languages.toml index 47155523..1ca40377 100644 --- a/languages.toml +++ b/languages.toml @@ -233,3 +233,15 @@ indent = { tab-width = 4, unit = " " } # comment-token = "--" # # indent = { tab-width = 2, unit = " " } + +[[language]] +name = "zig" +scope = "source.zig" +injection-regex = "zig" +file-types = ["zig"] +roots = ["build.zig"] +auto-format = true +comment-token = "//" + +language-server = { command = "zls" } +indent = { tab-width = 4, unit = " " } diff --git a/runtime/queries/zig/highlights.scm b/runtime/queries/zig/highlights.scm new file mode 100644 index 00000000..5a3d62dc --- /dev/null +++ b/runtime/queries/zig/highlights.scm @@ -0,0 +1,198 @@ +[ + (container_doc_comment) + (doc_comment) + (line_comment) +] @comment + +; field in top level decl, and in struct, union... +(ContainerField + (IDENTIFIER) @property + (SuffixExpr (IDENTIFIER) @type)? +) + +; error.OutOfMemory; +(SuffixExpr + "error" + "." + (IDENTIFIER) @constant +) + +; var x: IDENTIFIER +type: (SuffixExpr (IDENTIFIER) @type) + +; IDENTIFIER{} +constructor: (SuffixExpr (IDENTIFIER) @constructor) + +; fields +(FieldInit (IDENTIFIER) @property) + +; foo.bar.baz.function() calls +( + (SuffixOp + (IDENTIFIER) @function + ) + . + (FnCallArguments) +) + +; function() calls +( + ( + (IDENTIFIER) @function + ) + . + (FnCallArguments) +) + +; functionn decl +(FnProto + (IDENTIFIER) @function + (SuffixExpr (IDENTIFIER) @type)? + ("!")? @function.macro +) + +; function parameters and types +(ParamDecl + (IDENTIFIER) @variable.parameter + ":" + [ + (ParamType (SuffixExpr (IDENTIFIER) @type)) + (ParamType) + ] +) + +; switch +(SwitchItem + (SuffixExpr + "." + . + (IDENTIFIER) @constant + ) +) + +(INTEGER) @number + +(FLOAT) @number + +[ + (STRINGLITERAL) + (STRINGLITERALSINGLE) +] @string + +(CHAR_LITERAL) @string + +[ + "allowzero" + "volatile" + "anytype" + "anyframe" + (BuildinTypeExpr) +] @type.builtin + +(BreakLabel (IDENTIFIER) @label) +(BlockLabel (IDENTIFIER) @label) + +[ + "true" + "false" + "undefined" + "unreachable" + "null" +] @constant.builtin + +[ + "else" + "if" + "switch" + "for" + "while" + "return" + "break" + "continue" + "defer" + "errdefer" + "async" + "nosuspend" + "await" + "suspend" + "resume" + "try" + "catch" +] @keyword.control + +[ + "struct" + "enum" + "union" + "error" + "packed" + "opaque" + "test" + "usingnamespace" + "export" + "extern" + "const" + "var" + "comptime" + "threadlocal" +] @keyword + +[ + "pub" + "fn" +] @keyword.function + +; PrecProc +[ + "inline" + "noinline" + "asm" + "callconv" + "noalias" +] @attribute + +[ + (BUILTINIDENTIFIER) + "linksection" + "align" +] @function.builtin + +[ + (CompareOp) + (BitwiseOp) + (BitShiftOp) + (AdditionOp) + (MultiplyOp) + (PrefixOp) + "or" + "and" + "orelse" + "*" + "**" + "->" + "=>" + ".?" + ".*" + "=" +] @operator + +[ + ";" + "." + "," + ":" +] @punctuation.delimiter + +[ + ".." + "..." + "[" + "]" + "(" + ")" + "{" + "}" + (Payload "|") + (PtrPayload "|") + (PtrIndexPayload "|") +] @punctuation diff --git a/runtime/queries/zig/indents.toml b/runtime/queries/zig/indents.toml new file mode 100644 index 00000000..e119078b --- /dev/null +++ b/runtime/queries/zig/indents.toml @@ -0,0 +1,12 @@ +indent = [ + "block", + "match_block", + "arguments", + "parameters" +] + +outdent = [ + "}", + "]", + ")" +] -- cgit v1.2.3-70-g09d2 From d6a9c2c0f6f4af98146b52d1c886a1ca99d15676 Mon Sep 17 00:00:00 2001 From: CossonLeo Date: Sat, 28 Aug 2021 12:54:24 +0800 Subject: Add ui.menu text style (#664) * add menu text style * add ui.menu.text ui.info ui.info.text to book * change ui.menu.text to ui.menu * fix book's ui.menu--- book/src/themes.md | 3 +++ helix-term/src/ui/menu.rs | 7 +++++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/book/src/themes.md b/book/src/themes.md index f17195aa..0a4d58ad 100644 --- a/book/src/themes.md +++ b/book/src/themes.md @@ -91,6 +91,9 @@ Possible keys: | `ui.help` | | | `ui.text` | | | `ui.text.focus` | | +| `ui.info` | | +| `ui.info.text` | | +| `ui.menu` | | | `ui.menu.selected` | | | `ui.selection` | For selections in the editing area | | `ui.selection.primary` | | diff --git a/helix-term/src/ui/menu.rs b/helix-term/src/ui/menu.rs index a56cf19b..24dd3e61 100644 --- a/helix-term/src/ui/menu.rs +++ b/helix-term/src/ui/menu.rs @@ -259,8 +259,11 @@ impl Component for Menu { // TODO: required size should re-trigger when we filter items so we can draw a smaller menu fn render(&mut self, area: Rect, surface: &mut Surface, cx: &mut Context) { - let style = cx.editor.theme.get("ui.text"); - let selected = cx.editor.theme.get("ui.menu.selected"); + let theme = &cx.editor.theme; + let style = theme + .try_get("ui.menu") + .unwrap_or_else(|| theme.get("ui.text")); + let selected = theme.get("ui.menu.selected"); let scroll = self.scroll; -- cgit v1.2.3-70-g09d2