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

Updated how the mappings are done #54

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

Conversation

RichGuk
Copy link

@RichGuk RichGuk commented Aug 1, 2011

Hey,

I've created a patch to update how the mappings are done so you can override them with ease, for example you can now do things like

let g:ruby_debugger_create_default_mappings = 0

" Setup mappings for ruby-debugger.
nmap <leader>db <plug>ruby_debugger_breakpoint
nmap <leader>dv <plug>ruby_debugger_open_variables
nmap <leader>dm <plug>ruby_debugger_open_breakpoints
nmap <leader>dt <plug>ruby_debugger_open_frames
nmap <leader>ds <plug>ruby_debugger_step
nmap <leader>df <plug>ruby_debugger_finish
nmap <leader>dn <plug>ruby_debugger_next
nmap <leader>dc <plug>ruby_debugger_continue
nmap <leader>de <plug>ruby_debugger_exit
nmap <leader>dd <plug>ruby_debugger_remove_breakpoints

(I like to prefix all the commands with d)

It will now also make sure the mapping isn't already in use, I had a lot of conflicts with NERD Tree and Command-T.

@kibs
Copy link

kibs commented Aug 24, 2011

+1

@akemrir
Copy link

akemrir commented Sep 21, 2011

+1 thanks for help

@skwp
Copy link

skwp commented Jun 1, 2012

👍

@phantomwhale
Copy link

👍 - but looks like I'll be using @RichGuk's fork for now !

@phantomwhale
Copy link

Ok, after a little poking and attempting to merge the flag into a new fork, I notice that the g:ruby_debugger_no_maps flag is in the main master branch - so this pull request should really be closed now, as this flag fulfils the same intent ?

@RichGuk
Copy link
Author

RichGuk commented Mar 12, 2014

It does look like that's the case. I feel after 3 years that it's most likely not being merged anyhow ;)

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.

5 participants