Skip to content

Add option for case-insensitive omnicomplete#197

Merged
lervag merged 2 commits intolervag:masterfrom
anjiro:ci_autocomplete
Nov 6, 2021
Merged

Add option for case-insensitive omnicomplete#197
lervag merged 2 commits intolervag:masterfrom
anjiro:ci_autocomplete

Conversation

@anjiro
Copy link
Copy Markdown
Contributor

@anjiro anjiro commented Nov 5, 2021

My wiki files all have the first letter capitalized, which makes case-sensitive searching inconvenient. I added an option g:wiki_completion_case_sensitive to optionally do case-insensitive searching.

Copy link
Copy Markdown
Owner

@lervag lervag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Again I'm glad to see your thoughtful and high quality contribution. I do have some comments, but they are mostly neutral. I'd like to hear your thoughts, but we might conclude to merge without any further updates.

call map(l:candidates, 'v:val.word')
if !s:completer_wikilink.is_anchor
return filter(l:candidates, 'stridx(v:val, l:input) == 0')
return s:filter_candidates(l:candidates, '^' . l:input)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, that's smart!

lervag added a commit that referenced this pull request Nov 6, 2021
@lervag lervag merged commit 01bc076 into lervag:master Nov 6, 2021
@lervag
Copy link
Copy Markdown
Owner

lervag commented Nov 6, 2021

Thanks, I've merged now!

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.

2 participants