Skip to content

[vim keymap] Add actionArgs for command 'i' 'I'#1525

Closed
lynschinzer wants to merge 2 commits intocodemirror:masterfrom
lynschinzer:vim_restruct_I_cmd
Closed

[vim keymap] Add actionArgs for command 'i' 'I'#1525
lynschinzer wants to merge 2 commits intocodemirror:masterfrom
lynschinzer:vim_restruct_I_cmd

Conversation

@lynschinzer
Copy link
Copy Markdown
Contributor

This pull request is responding to the received suggestions from #1521.

actionArgs.insertAt can be later used for recording cursor behaviors for latest enterInsertMode commands.

@mightyguava
Copy link
Copy Markdown
Contributor

Looks good. Nit: in general, prefer == over ===.

@lynschinzer
Copy link
Copy Markdown
Contributor Author

Nit: in general, prefer == over ===.

After viewing the benchmarks from
http://jsperf.com/comparison-of-comparisons

I kinda agree with you.
In Chrome, === is no longer faster than ==

@lynschinzer
Copy link
Copy Markdown
Contributor Author

That should do it.

@mightyguava
Copy link
Copy Markdown
Contributor

Looks good. Thanks.

marijnh added a commit that referenced this pull request May 14, 2013
@marijnh
Copy link
Copy Markdown
Member

marijnh commented May 14, 2013

Merged. Stray character removed in attached patch.

@marijnh marijnh closed this May 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants