-
Notifications
You must be signed in to change notification settings - Fork 154
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
feat: cycle completions #12
Conversation
Thank you! I'd prefer config driven like |
@mdjohns , could you also update the readme in this PR, because the username in readme to install the plugin is incorrect {
'saghen/blink.cmp', -- should be "Saghen/blink.cmp"
lazy = false, -- lazy loading handled internally
-- optional: provides snippets for the snippet source
dependencies = 'rafamadriz/friendly-snippets',
-- use a release tag to download pre-built binaries
version = 'v0.*',
-- OR build from source, requires nightly: https://rust-lang.github.io/rustup/concepts/channels.html#working-with-nightly-rust
-- build = 'cargo build --release',
opts = {
highlight = {
-- sets the fallback highlight groups to nvim-cmp's highlight groups
-- useful for when your theme doesn't support blink.cmp
-- will be removed in a future release, assuming themes add support
use_nvim_cmp_as_default = true,
},
-- set to 'mono' for 'Nerd Font Mono' or 'normal' for 'Nerd Font'
-- adjusts spacing to ensure icons are aligned
nerd_font_variant = 'normal',
-- experimental auto-brackets support
-- accept = { auto_brackets = { enabled = true } }
-- experimental signature help support
-- trigger = { signature_help = { enabled = true } }
}
}
|
12ce4bb
to
2fc28c3
Compare
2fc28c3
to
7539d82
Compare
@Saghen done! i also updated the readme to reflect the change. |
@shubham-cpp these URLs aren't case sensitive: The lowercase URL works fine in my config. |
Thank you! |
resolves #8.
calling
select_prev
at the beginning of the completion list will now cycle to the end.calling
select_next
at the end of the completion list will now cycle back to the beginning.it may make sense to make this config-driven, let me know what you'd prefer!