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

Include Stop and Start Spin commands #38

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

holdenmh1214
Copy link

@holdenmh1214 holdenmh1214 commented Apr 27, 2020

Saw Zach was playing around with nim and decided to go down a 4hr rabbit hole to learn about it and add this feature.

Don't hate me

New commands:
Stop
Start
Slower
Faster

This commit addresses issue #3

Fixed issue where commands were getting executed before hitting 'enter'
@oakes
Copy link
Owner

oakes commented Apr 27, 2020

HOLDEN!! long time no see man. yeah i am really into nim these days. funny enough, this is a feature i plan on adding directly to paravim -- custom commands like :whatever that run arbitrary code you specify. you did it without needing to modify paravim, impressive. you hacked a hack. more accurately, a hack²

@holdenmh1214
Copy link
Author

ZACH!!!! Yeah man it has been awhile since our fight club days.
I was just starting to dive into Rust until I checked your repos. Nim is interesting.
Yeah, this hack seemed like the simplest way of adding those features without diving very far into paravim.

This block made it possible:
`pararules.query(vim.session, vim.rules.getVim).mode

I then added a log to see what the modes were for each state of paravim. Once I had those, the rest was really easy.

@holdenmh1214
Copy link
Author

@oakes I added a "hammer_time" command.

Now when you enter the command "stop" and and then enter "hammer_time" the background image will change to MC Hammer.
Because why not

@holdenmh1214 holdenmh1214 mentioned this pull request Apr 29, 2020
@oakes
Copy link
Owner

oakes commented Apr 30, 2020

image

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.

2 participants