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

Implemented command mode, $, 0, ^, gg, G, plus inline search fx and Fx #77

Closed
wants to merge 12 commits into from
Closed

Conversation

rashwell
Copy link

@rashwell rashwell commented Dec 2, 2015

Looking over the changes I realize I might have to watch the formatting in things like the package.json. I don't understand how mine was formatted so differently. But I am also very new to git so I am not really sure how to selectively do stuff. But it was suggested to me to make this pull request so that the logic for 0 vs ^ could be in there. To keep this to a single commit, the fx and Fx are in here as well. I am hoping to chat with the major directors here though on the approach for thing like f and t commands, as you will see in my commit, I created sort of a duo key handler since it is different that two key commands. But there are a lot of ways to approach this. I have ideas, but right now this extension is so early, it might need a lot of refactoring, as we puzzle through this.

@jpoon
Copy link
Member

jpoon commented Dec 2, 2015

.. package.json. I don't understand how mine was formatted so differently.

Looks like you may have added lodash as a dev dependency. I have found that after doing an npm install --save/save-dev, it re-formats the package.json. You can commit certain parts of a file using git commit -i

">>" : () => { vscode.commands.executeCommand("editor.action.indentLines"); },
"<<" : () => { vscode.commands.executeCommand("editor.action.outdentLines"); },
"dd" : () => { vscode.commands.executeCommand("editor.action.deleteLines"); },
"dw" : () => { vscode.commands.executeCommand("deleteWordRight"); },
"db" : () => { vscode.commands.executeCommand("deleteWordLeft"); },
"gg" : () => { vscode.commands.executeCommand("cursorTop"); },
"G" : () => { vscode.commands.executeCommand("cursorBottom"); },
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will conflict with #71

@rashwell rashwell closed this Dec 2, 2015
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