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 commands support for 'gg' and 'G' #71

Merged
merged 1 commit into from
Dec 2, 2015
Merged

Conversation

liushuping
Copy link
Contributor

Add navigation commands support:

  • 'gg': go to fist line, on first non-blank character
  • 'G': go to last line, on first non-blank character

@mmcdole
Copy link

mmcdole commented Dec 1, 2015

👍

@jpoon
Copy link
Member

jpoon commented Dec 1, 2015

Would you mind adding tests? See this as a reference.

Regressions are starting to become an issue :(

@liushuping
Copy link
Contributor Author

Tests have been added and functions are renamed to 'firstLineNonBlanChar' and 'lastLineNonBlanChar' to differentiate them from document begin and end functions. Please review.

@jpoon
Copy link
Member

jpoon commented Dec 2, 2015

One last thing, can you squash your commits: git rebase -i HEAD~4. Thanks.

…haracter; 'G': go to last line, on first non-blank character
@liushuping
Copy link
Contributor Author

Update: commits are squashed to a single one

jpoon added a commit that referenced this pull request Dec 2, 2015
Add commands support for 'gg' and 'G'
@jpoon jpoon merged commit 6b36f2b into VSCodeVim:master Dec 2, 2015
@jpoon
Copy link
Member

jpoon commented Dec 2, 2015

👍

@liushuping liushuping deleted the cmd_g branch December 3, 2015 12:24
This pull request was closed.
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