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

Add a playback command to execute recordings. #1

Merged
merged 1 commit into from Feb 24, 2012

Conversation

aszlig
Copy link
Contributor

@aszlig aszlig commented Feb 24, 2012

Previously, recordings were saved to a normal keybinding, which could
possibly shadowed by existing keybindings and thus the recorded key
events couldn't be run.

This commit fixes this by adding a playback command, which works much
like vi's @ command (which it is bound to by default).

For example:
qa -> Go into record mode and save to register 'a'
... do stuff ...
q -> Stop recording
@A -> Play back register 'a'

Previously, recordings were saved to a normal keybinding, which could
possibly shadowed by existing keybindings and thus the recorded key
events couldn't be run.

This commit fixes this by adding a playback command, which works much
like vi's @ command (which it is bound to by default).

For example:
qa -> Go into record mode and save to register 'a'
... do stuff ...
q -> Stop recording
@A -> Play back register 'a'
@jordansissel
Copy link
Owner

Hmm, good idea.

jordansissel added a commit that referenced this pull request Feb 24, 2012
Add a playback command to execute recordings.
@jordansissel jordansissel merged commit 8c17e31 into jordansissel:master Feb 24, 2012
@jordansissel
Copy link
Owner

I haven't tested this yet, but will soon, code looks good otherwise. :)

@aszlig
Copy link
Contributor Author

aszlig commented Feb 24, 2012

Added two more commits to this branch which add the command to the POD file and the example config.

@jordansissel
Copy link
Owner

Merged those commits too, thanks!

@aszlig
Copy link
Contributor Author

aszlig commented Feb 24, 2012

Whoops, sorry for the noise, one more commit to go, because I somehow forgot about "keynavrc".
And thanks for merging so fast (faster than I expected, because I hoped I could add the doc/example config stuff before the merge).

mgsloan pushed a commit to mgsloan/keynav that referenced this pull request Aug 26, 2017
Merge marekjm's fork (mostly adding install target to Makefile)
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.

None yet

2 participants