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

The character input only expects lower case letters #19

Open
123subhadeep opened this issue Feb 28, 2020 · 2 comments · May be fixed by #26
Open

The character input only expects lower case letters #19

123subhadeep opened this issue Feb 28, 2020 · 2 comments · May be fixed by #26

Comments

@123subhadeep
Copy link

Let's say I would like to jump to the beginning of the word "Success".
Entering "S" after doing + j doesn't work. The plugin seems to be always expecting lower case characters, even when the word itself starts with capital letter.

It would be better to have that fixed, there are a couple of ways to solve it:

  1. Do case sensitive search, eg: if S is entered only match words starting with S, similarly s only matches words starting with s.
  2. Do smartcase search, eg: S matches only S whereas s matches both s and S
@123subhadeep
Copy link
Author

Seems like removing ".downcase" in these two lines should do the job: https://github.com/schasse/tmux-jump/blob/master/scripts/tmux-jump.rb#L136,L138

@ywpkwon
Copy link

ywpkwon commented Mar 30, 2020

How about the special characters like .$%^ and so on? Let say there is some path like ./.config/i3/xxx/bbb/ccc.ddd on a screen and I want a quick copy of it. I think I would want to jump to ..

Lenbok added a commit to Lenbok/tmux-jump that referenced this issue Jul 9, 2020
If character is uppercase, use case-sensitive matching
Otherwise use case-insensitive matching

Fixes: schasse#19
@Lenbok Lenbok linked a pull request Jul 9, 2020 that will close this issue
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 a pull request may close this issue.

2 participants