diff options
author | Blaž Hrastnik | 2021-03-02 06:44:09 +0000 |
---|---|---|
committer | Blaž Hrastnik | 2021-03-02 06:44:09 +0000 |
commit | 8f4ff4c646988c377218c08c851297e68e860c65 (patch) | |
tree | 984f5293b6f0f6e293b3c2c75def00b39b008030 | |
parent | 32f9a2d1d6fe4955cffffa71bbdfc5a43a6f0c9c (diff) |
editor: We still want to be able to calculate cursor pos.
-rw-r--r-- | helix-term/src/commands.rs | 6 | ||||
-rw-r--r-- | helix-term/src/ui/editor.rs | 9 | ||||
-rw-r--r-- | helix-view/src/editor.rs | 12 |
3 files changed, 15 insertions, 12 deletions
diff --git a/helix-term/src/commands.rs b/helix-term/src/commands.rs index 0d07059f..8c46a1cf 100644 --- a/helix-term/src/commands.rs +++ b/helix-term/src/commands.rs @@ -1119,8 +1119,7 @@ pub fn completion(cx: &mut Context) { cx.callback = Some(Box::new( move |compositor: &mut Compositor, editor: &mut Editor| { - let area = tui::layout::Rect::default(); // TODO: unused remove from cursor_position - if let Some(mut pos) = compositor.cursor_position(area, editor) { + if let Some(mut pos) = editor.cursor_position() { pos.row += 1; // shift down by one row menu.set_position(pos); }; @@ -1181,8 +1180,7 @@ pub fn hover(cx: &mut Context) { cx.callback = Some(Box::new( move |compositor: &mut Compositor, editor: &mut Editor| { - let area = tui::layout::Rect::default(); // TODO: unused remove from cursor_position - if let Some(mut pos) = compositor.cursor_position(area, editor) { + if let Some(mut pos) = editor.cursor_position() { pos.row += 1; // shift down by one row popup.set_position(pos); }; diff --git a/helix-term/src/ui/editor.rs b/helix-term/src/ui/editor.rs index 2aa1d469..f1860a27 100644 --- a/helix-term/src/ui/editor.rs +++ b/helix-term/src/ui/editor.rs @@ -433,14 +433,7 @@ impl Component for EditorView { // Mode::Insert => write!(stdout, "\x1B[6 q"), // mode => write!(stdout, "\x1B[2 q"), // }; - // let view = editor.view(); - // let cursor = view.doc.selection().cursor(); - - // if let Some(mut pos) = view.screen_coords_at_pos(view.doc.text().slice(..), cursor) { - // pos.col += view.area.x as usize + area.x as usize + OFFSET as usize; - // pos.row += view.area.y as usize + area.y as usize; - // return Some(pos); - // } + // return editor.cursor_position() // It's easier to just not render the cursor and use selection rendering instead. None diff --git a/helix-view/src/editor.rs b/helix-view/src/editor.rs index c5597a34..c072d76f 100644 --- a/helix-view/src/editor.rs +++ b/helix-view/src/editor.rs @@ -74,4 +74,16 @@ impl Editor { pub fn view_mut(&mut self) -> &mut View { self.tree.get_mut(self.tree.focus) } + + pub fn cursor_position(&self) -> Option<helix_core::Position> { + const OFFSET: u16 = 7; // 1 diagnostic + 5 linenr + 1 gutter + let view = self.view(); + let cursor = view.doc.selection().cursor(); + if let Some(mut pos) = view.screen_coords_at_pos(view.doc.text().slice(..), cursor) { + pos.col += view.area.x as usize + OFFSET as usize; + pos.row += view.area.y as usize; + return Some(pos); + } + None + } } |