Skip to content

Updated how the mappings are done#54

Open
RichGuk wants to merge 2 commits into
astashov:masterfrom
RichGuk:master
Open

Updated how the mappings are done#54
RichGuk wants to merge 2 commits into
astashov:masterfrom
RichGuk:master

Conversation

@RichGuk

@RichGuk RichGuk commented Aug 1, 2011

Copy link
Copy Markdown

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

kibs commented Aug 24, 2011

Copy link
Copy Markdown

+1

@akemrir

akemrir commented Sep 21, 2011

Copy link
Copy Markdown

+1 thanks for help

@skwp

skwp commented Jun 1, 2012

Copy link
Copy Markdown

👍

@phantomwhale

Copy link
Copy Markdown

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

@phantomwhale

Copy link
Copy Markdown

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

RichGuk commented Mar 12, 2014

Copy link
Copy Markdown
Author

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