b765c17 · helix · git.iko.soy

git.iko.soy/helix

Hacky undo/redo integration.

commitb765c17896632e27857c7707a9d3d96bd49b8899 authorBlaž Hrastnik <blaz@mxxn.io> dateTue 6 Oct 2020, 14:44 (6 years ago) parent0926904

4 files changed+36−39

compare with master

unifiedsplit

helix-core/src/state.rs+5−1 helix-core/src/transaction.rs+7 helix-term/src/editor.rs−28 helix-view/src/commands.rs+24−10

helix-core/src/state.rs+5−1

@@ -1,6 +1,6 @@

11use crate::graphemes::{nth_next_grapheme_boundary, nth_prev_grapheme_boundary, RopeGraphemes};

22use crate::syntax::LOADER;

3−use crate::{Position, Range, Rope, RopeSlice, Selection, Syntax};

3+use crate::{ChangeSet, Position, Range, Rope, RopeSlice, Selection, Syntax};

44use anyhow::Error;

66use std::path::PathBuf;

@@ -25,6 +25,8 @@ pub struct State {

2525

2626 //

2727 pub syntax: Option<Syntax>,

28+ pub changes: Option<ChangeSet>,

29+ pub old_state: Option<(Rope, Selection)>,

2830}

2931

3032#[derive(Copy, Clone, PartialEq, Eq)]

@@ -50,6 +52,8 @@ impl State {

5052 mode: Mode::Normal,

5153 restore_cursor: false,

5254 syntax: None,

55+ changes: None,

56+ old_state: None,

5357 }

5458 }

5559

helix-core/src/transaction.rs+7

@@ -369,6 +369,13 @@ impl Transaction {

369369 return false;

370370 }

371371

372+ // Compose this transaction with the previous one

373+ let old_changes = state.changes.take();

374+ state.changes = Some(old_changes.map_or_else(

375+ || self.changes.clone(),

376+ |changes| changes.compose(self.changes.clone()).unwrap(),

377+ ));

378+

372379 if let Some(syntax) = &mut state.syntax {

373380 // TODO: no unwrap

374381 syntax.update(&old_doc, &state.doc, &self.changes).unwrap();

helix-term/src/editor.rs−28

@@ -426,31 +426,3 @@ impl Editor {

426426// TODO: scope matching: biggest union match? [string] & [html, string], [string, html] & [ string, html]

427427// can do this by sorting our theme matches based on array len (longest first) then stopping at the

428428// first rule that matches (rule.all(|scope| scopes.contains(scope)))

429−//

430−// let visual_x = 0;

431−// let line = ?;

432−// for span in spans {

433−// start(scope) => scopes.push(scope)

434−// span =>

435−// let text = rope.slice(span.start..span.end);

436−// let style = calculate_style(scopes);

437−// for each grapheme in text.graphemes() {

438−// // if newline += lines, continue

439−//

440−// if state.selection.ranges().any(|range| range.contains(char_index)) {

441−// if exactly on cursor {

442−// }

443−// if on primary cursor? {

444−// }

445−// modify style temporarily

446−// }

447−//

448−// // if in bounds

449−//

450−// // if tab, draw tab width

451−// // draw(visual_x, line, grapheme, style)

452−// // increment visual_x by grapheme_width(grapheme)

453−// // increment char_index by grapheme.len_chars()

454−// }

455−// end => scopes.pop()

456−// }

helix-view/src/commands.rs+24−10

@@ -3,7 +3,7 @@ use helix_core::{

33 regex::Regex,

44 selection,

55 state::{Direction, Granularity, Mode, State},

6− Range, Selection, Tendril, Transaction,

6+ ChangeSet, Range, Selection, Tendril, Transaction,

77};

88use once_cell::sync::Lazy;

@@ -253,14 +253,16 @@ pub fn collapse_selection(view: &mut View, _count: usize) {

253253 .transform(|range| Range::new(range.head, range.head))

254254}

255255

256−// insert mode:

257−// first we calculate the correct cursors/selections

258−// then we just append at each cursor

259−// lastly, if it was append mode we shift cursor by 1?

256+fn enter_insert_mode(view: &mut View) {

257+ view.state.mode = Mode::Insert;

260258

259+ // HAXX

260+ view.state.changes = Some(ChangeSet::new(view.state.doc()));

261+ view.state.old_state = Some((view.state.doc().clone(), view.state.selection.clone()));

262+}

261263// inserts at the start of each selection

262264pub fn insert_mode(view: &mut View, _count: usize) {

263− view.state.mode = Mode::Insert;

265+ enter_insert_mode(view);

264266

265267 view.state.selection = view

266268 .state

@@ -270,7 +272,7 @@ pub fn insert_mode(view: &mut View, _count: usize) {

270272

271273// inserts at the end of each selection

272274pub fn append_mode(view: &mut View, _count: usize) {

273− view.state.mode = Mode::Insert;

275+ enter_insert_mode(view);

274276 view.state.restore_cursor = true;

275277

276278 // TODO: as transaction

@@ -303,21 +305,21 @@ fn selection_lines(state: &State) -> Vec<usize> {

303305

304306// I inserts at the start of each line with a selection

305307pub fn prepend_to_line(view: &mut View, count: usize) {

306− view.state.mode = Mode::Insert;

308+ enter_insert_mode(view);

307309

308310 move_line_start(view, count);

309311}

310312

311313// A inserts at the end of each line with a selection

312314pub fn append_to_line(view: &mut View, count: usize) {

313− view.state.mode = Mode::Insert;

315+ enter_insert_mode(view);

314316

315317 move_line_end(view, count);

316318}

317319

318320// o inserts a new line after each line with a selection

319321pub fn open_below(view: &mut View, _count: usize) {

320− view.state.mode = Mode::Insert;

322+ enter_insert_mode(view);

321323

322324 let lines = selection_lines(&view.state);

323325

@@ -356,6 +358,18 @@ pub fn open_below(view: &mut View, _count: usize) {

356358pub fn normal_mode(view: &mut View, _count: usize) {

357359 view.state.mode = Mode::Normal;

358360

361+ if let Some(changes) = view.state.changes.take() {

362+ // Instead of doing this messy merge we could always commit, and based on transaction

363+ // annotations either add a new layer or compose into the previous one.

364+ let transaction = Transaction::from(changes).with_selection(view.state.selection().clone());

365+ let (doc, selection) = view.state.old_state.take().unwrap();

366+ let mut old_state = State::new(doc);

367+ old_state.selection = selection;

368+

369+ // TODO: take transaction by value?

370+ view.history.commit_revision(&transaction, &old_state);

371+ }

372+

359373 // if leaving append mode, move cursor back by 1

360374 if view.state.restore_cursor {

361375 let text = &view.state.doc.slice(..);