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

Pop-up documentation should not block the view to the cursor #307

Closed
malte-v opened this issue Jun 18, 2021 · 2 comments · Fixed by #322
Closed

Pop-up documentation should not block the view to the cursor #307

malte-v opened this issue Jun 18, 2021 · 2 comments · Fixed by #322
Labels
A-documentation Area: Documentation improvements C-enhancement Category: Improvements E-help-wanted Call for participation: Extra attention is needed

Comments

@malte-v
Copy link
Contributor

malte-v commented Jun 18, 2021

hx
That should make the issue clear. I can't see what I'm editing because the documentation is in the way.

@pickfire
Copy link
Contributor

I see this issue in kakoune quite often, so most of the time I just disable the popup because of this.

@pickfire pickfire added A-documentation Area: Documentation improvements C-enhancement Category: Improvements E-help-wanted Call for participation: Extra attention is needed labels Jun 19, 2021
@archseer
Copy link
Member

Yeah I was planning on placing the docs above if they get too close to the prompt. (Easy reasoning would be if the cursor is past the middle of the view: nvim-lua/completion-nvim@f099367#diff-13ac7f54ec22903ec9e2cd8a4874aceed4b78ece9e479d165d68631ecf4a3373R205)

Another option is a popup next to the completion, similar to compe (https://github.com/hrsh7th/nvim-compe#demo) but I find that harder to read than a pane at the top/bottom.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-documentation Area: Documentation improvements C-enhancement Category: Improvements E-help-wanted Call for participation: Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants