git.iko.soy/helix
on master help sign in
Implement vertical split calculations.
commit5e73f83efa406a6989fff6077b90dd77a7328b36 authorBlaž Hrastnik <blaz@mxxn.io> dateTue 9 Feb 2021, 15:59 (6 years ago) parentd4b85ce
5 files changed+39−11
compare with master
unifiedsplit
TODO.md+2−1 helix-term/src/ui/editor.rs+7−7 helix-view/src/editor.rs+4−1 helix-view/src/tree.rs+25−1 helix-view/src/view.rs+1−1
TODO.md+2−1
@@ -10,8 +10,9 @@
10101
1111- [ ] selection mode
1212- [x] % for whole doc selection
13−- [ ] vertical splits
13+- [x] vertical splits
1414- [ ] input counts (30j)
15+- [ ] respect view fullscreen flag
1516- [ ] retain horiz when moving vertically
1617- [ ] update lsp on redo/undo
1718- [ ] Implement marks (superset of Selection/Range)
helix-term/src/ui/editor.rs+7−7
@@ -42,8 +42,8 @@ impl EditorView {
4242 viewport.x + OFFSET,
4343 viewport.y,
4444 viewport.width - OFFSET,
45− viewport.height - 2,
46− ); // - 2 for statusline and prompt
45+ viewport.height - 1,
46+ ); // - 1 for statusline
4747 self.render_buffer(view, area, surface, theme, is_focused);
4848
4949 // clear with background color
@@ -52,7 +52,7 @@ impl EditorView {
5252
5353 let area = Rect::new(
5454 viewport.x,
55− viewport.y + viewport.height - 2,
55+ viewport.y + viewport.height - 1,
5656 viewport.width,
5757 1,
5858 );
@@ -301,9 +301,8 @@ impl Component for EditorView {
301301 fn handle_event(&mut self, event: Event, cx: &mut Context) -> EventResult {
302302 match event {
303303 Event::Resize(width, height) => {
304− // TODO: simplistic ensure cursor in view for now
305− // TODO: loop over views
306− cx.editor.tree.resize(Rect::new(0, 0, width, height));
304+ // HAXX: offset the render area height by 1 to account for prompt/commandline
305+ cx.editor.tree.resize(Rect::new(0, 0, width, height - 1));
307306 // TODO: restore view.ensure_cursor_in_view();
308307 EventResult::Consumed(None)
309308 }
@@ -353,10 +352,11 @@ impl Component for EditorView {
353352 }
354353 }
355354
356− fn render(&self, area: Rect, surface: &mut Surface, cx: &mut Context) {
355+ fn render(&self, mut area: Rect, surface: &mut Surface, cx: &mut Context) {
357356 // SAFETY: we cheat around the view_mut() borrow because it doesn't allow us to also borrow
358357 // theme. Theme is immutable mutating view won't disrupt theme_ref.
359358 let theme_ref = unsafe { &*(&cx.editor.theme as *const Theme) };
359+
360360 for (view, is_focused) in cx.editor.tree.views() {
361361 // TODO: use parent area
362362 self.render_view(view, view.area, surface, theme_ref, is_focused);
helix-view/src/editor.rs+4−1
@@ -15,10 +15,13 @@ pub struct Editor {
1515}
1616
1717impl Editor {
18− pub fn new(area: tui::layout::Rect) -> Self {
18+ pub fn new(mut area: tui::layout::Rect) -> Self {
1919 let theme = Theme::default();
2020 let language_servers = helix_lsp::Registry::new();
2121
22+ // HAXX: offset the render area height by 1 to account for prompt/commandline
23+ area.height -= 1;
24+
2225 Self {
2326 tree: Tree::new(area),
2427 should_close: false,
helix-view/src/tree.rs+25−1
@@ -156,7 +156,31 @@ impl Tree {
156156 container.area = area;
157157
158158 match container.layout {
159− Layout::Vertical => unimplemented!(),
159+ Layout::Vertical => {
160+ let len = container.children.len();
161+
162+ let height = area.height / len as u16;
163+
164+ let mut child_y = area.y;
165+
166+ for (i, child) in container.children.iter().enumerate() {
167+ let mut area = Rect::new(
168+ container.area.x,
169+ child_y,
170+ container.area.width,
171+ height,
172+ );
173+ child_y += height;
174+
175+ // last child takes the remaining width because we can get uneven
176+ // space from rounding
177+ if i == len - 1 {
178+ area.height = container.area.y + container.area.height - area.y;
179+ }
180+
181+ self.stack.push((*child, area));
182+ }
183+ }
160184 Layout::Horizontal => {
161185 let len = container.children.len();
162186
helix-view/src/view.rs+1−1
@@ -51,7 +51,7 @@ impl View {
5151 /// Calculates the last visible line on screen
5252 #[inline]
5353 pub fn last_line(&self) -> usize {
54− let viewport = Rect::new(6, 0, self.area.width, self.area.height - 2); // - 2 for statusline and prompt
54+ let viewport = Rect::new(6, 0, self.area.width, self.area.height - 1); // - 1 for statusline
5555 std::cmp::min(
5656 self.first_line + (viewport.height as usize),
5757 self.doc.text().len_lines() - 1,