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

g] doesn't work? #37

Closed
goodboy opened this issue Jan 3, 2017 · 5 comments
Closed

g] doesn't work? #37

goodboy opened this issue Jan 3, 2017 · 5 comments

Comments

@goodboy
Copy link
Contributor

goodboy commented Jan 3, 2017

The binding of g] to :tselect doesn't seem to work after installing this plugin?

Is this expected?
I'd love to have both since I use tags quite often.

@goodboy
Copy link
Contributor Author

goodboy commented Jan 3, 2017

Ahh I found where you document this:

By default, these let you move around using the [[ and ]] keys.  The following
sequences are also mapped with these keys:                                    
                                                                              
  g]    Move forward to an increased indent level                             
  g[    Move backward to a decreased indent level                             
                                                                              
Default:                                                                      
                                                                              
  let g:braceless_jump_prev_key = '['                                         
  let g:braceless_jump_next_key = ']'                                         

So my question now is how do you disable these additional mappings?

@tweekmonster
Copy link
Owner

Set them to empty strings. In hindsight, these should've been disabled by default.

@tweekmonster
Copy link
Owner

Actually you're talking about just disabling g[ and g]. Setting those variables to empty strings will also disable [[ and ]]. But, with the other issue you brought up in #38, you probably still want to do that.

@goodboy
Copy link
Contributor Author

goodboy commented Jan 4, 2017

@tweekmonster ok so then how do I access the jump_x_key behavior whilst also avoiding the conflict with g]?

Maybe there's a better default mapping for what [[ and ]] (]k for key?) is currently doing and maybe it's better to just avoid duplicate mappings for the same behavior (i.e. rip out g[ and g])?

@goodboy
Copy link
Contributor Author

goodboy commented Jan 4, 2017

@tweekmonster I removed the duplicate mappings for g and now everything works as expected.

If I also remove it from the docs will you take a PR?

goodboy pushed a commit to goodboy/braceless.vim that referenced this issue Jan 4, 2017
These motions are redundant (since [[ and ]] are already used) and
also mask other standard mappings (eg. g] normally is shorthand for
:tselect). Adjust the docs to match.

Fixes tweekmonster#37
goodboy pushed a commit to goodboy/braceless.vim that referenced this issue Jan 8, 2017
The use of `g]` masks the standard mapping for `:tselect`.
Disable the indent jumping feature by default.
Indicate how to re-enable it in the docs.

Fixes tweekmonster#37
tweekmonster pushed a commit that referenced this issue Jan 8, 2017
The use of `g]` masks the standard mapping for `:tselect`.
Disable the indent jumping feature by default.
Indicate how to re-enable it in the docs.

Fixes #37
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