Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vim go to and add new line above/below (o | O) leaves flashing block (includes pr for fix) #5544

Closed
goofiw opened this issue Aug 25, 2018 · 2 comments
Labels

Comments

@goofiw
Copy link

goofiw commented Aug 25, 2018

Reproable on the demo site. I made a pull request that fixes the issue.

Link to PR

Basically, the runInOp is wrapped around the addNewLineAndIndent function so the new line ends up being created and moved to after we go into insert mode which is muddling things up. I just wrapped the enterInsertMode function in a timeout to push it to the next run loop so all the operations should have finished.

@goofiw goofiw changed the title Vim go to and add new line above/below (o | O) leaves flashing block Vim go to and add new line above/below (o | O) leaves flashing block [vim] Aug 25, 2018
@goofiw goofiw changed the title Vim go to and add new line above/below (o | O) leaves flashing block [vim] Vim go to and add new line above/below (o | O) leaves flashing block (vim) Aug 25, 2018
@goofiw goofiw changed the title Vim go to and add new line above/below (o | O) leaves flashing block (vim) Vim go to and add new line above/below (o | O) leaves flashing block Aug 25, 2018
@goofiw goofiw changed the title Vim go to and add new line above/below (o | O) leaves flashing block Vim go to and add new line above/below (o | O) leaves flashing block (includes pr for fix) Aug 25, 2018
@goofiw
Copy link
Author

goofiw commented Aug 26, 2018

Update: previous fix failed some tests (mainly replaying macros didn't work quite right) new pr:
#5547

I don't love the solution of moving the 'go to insert mode' into a timeout to push it off to the next run loop, but it works.

@adrianheine adrianheine added the vim label Sep 7, 2018
@marijnh
Copy link
Member

marijnh commented Jul 19, 2022

The vim code is now being maintained at https://github.com/replit/codemirror-vim/ . If this is still important, check whether there is an issue open for it there.

@marijnh marijnh closed this as completed Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants