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

Visual mode #24

Open
gchp opened this issue May 29, 2014 · 6 comments
Open

Visual mode #24

gchp opened this issue May 29, 2014 · 6 comments

Comments

@gchp
Copy link
Contributor

gchp commented May 29, 2014

Again, just another question.

In the normal mode file, there is the following:

        case termbox.KeyCtrlV:
            // TODO: Start visual selection
            return

Just want to get some clarification on this before starting work on it. Should CtrlV start regular visual selection? My impression would be that pressing v should start it, without the need for the control key.

@gchp
Copy link
Contributor Author

gchp commented May 29, 2014

In vim CtrlV starts rectangular selection, if I'm not mistaken

@kisielk
Copy link
Owner

kisielk commented May 29, 2014

Right on both counts. I think that case was just already in the switch so I added a comment as to what it should eventually do.

@gchp
Copy link
Contributor Author

gchp commented May 29, 2014

Cool, just wanted to double check, thanks!

If there's anything more pressing/important that you'd like worked on by way, just let me know. I'm really enjoying working on this project, so I'll work on whatever is most helpful for you guys.

@kisielk
Copy link
Owner

kisielk commented May 29, 2014

Nothing in particular. We really appreciate your contributions, so just keep doing what you're doing. If you have anything you're not sure about or want to address some higher-level things just file an issue to discuss it 👍

@gchp gchp mentioned this issue May 30, 2014
@gchp
Copy link
Contributor Author

gchp commented May 30, 2014

Just to note, there is an issue with clearing the selection when exiting visual mode. To fix this, we need to trigger a redraw inside the visualMode.Exit() method. This will need to wait until the redraw issue has been solved. Just wanted to record it here so it didn't get lost.

@kisielk
Copy link
Owner

kisielk commented May 30, 2014

I fixed it in 6e45cfd, left a comment there.

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

No branches or pull requests

2 participants