Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add some kyebinding #5

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

Conversation

skanehira
Copy link
Contributor

No description provided.

@MichaelMure
Copy link
Owner

Could you explain why you choose those keys ?

@skanehira
Copy link
Contributor Author

@MichaelMure Added keys just like a Vim keybinding.
I think this is a useful keybinding for many Vim users :)

@MichaelMure
Copy link
Owner

I'm a bit confused.

https://hea-www.harvard.edu/~fine/Tech/vi.html say that:

  • Ctrl+U and Ctrl+D are going up and down half a screen, not go to beginning or end
  • g is not used by default
  • G is going to a specified line, which make more sense to me, even though the concept of line is a bit blurry here (source line ? rendered line ?)

@afranke what do you think ?

@afranke
Copy link

afranke commented Jan 24, 2020

For page up/down in vim I do ctrl+f/ctrl+b. I never use the u/d ones. I use gg to go back to beginning of file all the time, and G to go to end of file. These should map well to mdr.

Of course I also use 42G to go to line 42. What that should do in mdr is show the output that matches the 42nd line from the source file in the middle of the screen, I reckon.

@MichaelMure
Copy link
Owner

Of course I also use 42G to go to line 42. What that should do in mdr is show the output that matches the 42nd line from the source file in the middle of the screen, I reckon.

The mapping from source line to output line is entirely lost when going through the parser.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants