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

case_insensitive Isn't working #74

Closed
carlos-algms opened this issue May 15, 2024 · 3 comments · Fixed by #76
Closed

case_insensitive Isn't working #74

carlos-algms opened this issue May 15, 2024 · 3 comments · Fixed by #76

Comments

@carlos-algms
Copy link

Describe the bug
Setting the case_insensitive = false, on the opts key and then running commands like: :HopChar1 doesn't seem to be working

To Reproduce
Steps to reproduce the behavior:

-- bootstrap lazy
        local lazypath = root .. "/plugins/lazy.nvim"
        if not vim.loop.fs_stat(lazypath) then
          vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
        end
        vim.opt.runtimepath:prepend(lazypath)

-- install hop.nvim
        local plugins = {
          { 
            "smoka7/hop.nvim", 
            opts = { 
               case_insensitive = false, 
            } 
          },
        }
        require("lazy").setup(plugins,{})
--- How You use hop...

:HopChar1

Expected behavior
It should respect the case_insensitive prop and only highlight the matching capital letters.

version (please complete the following information):

  • Nvim version: 0.9.5
  • hop.nvim version: * (latest)
@teddywaweru
Copy link

case_insensitive is in the config, not the opts table:

require("hop").setup { 

keys = 'etovxqpdygfblzhckisuran',
case_insensitive = false
 }
Though it is a bit awkward, since it'll apply to all hop autocmds...

@carlos-algms
Copy link
Author

the opts table should be passed to setup(...) automatically by lazy.

@smoka7
Copy link
Owner

smoka7 commented May 23, 2024

Checkout the linked PR to see if it works as expected.

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.

3 participants