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

util: add auto quickfix window height for errors [GH-592] #602

Merged
merged 1 commit into from
Nov 16, 2015

Conversation

fatih
Copy link
Owner

@fatih fatih commented Nov 15, 2015

Right now we are just calling cwindow without any arguments. By
default Vim opens a view with an height of 10. But this doesn't look
good if you have only on single error. It also takes display estate for
small screens.

This PR introduces a new feature, which automatically resizes the
quickfix error for the number of errors. However to prevent consuming
all the view, it's capped at 10 lines (the default Vim value).

We also have a new setting, g:go_quickfix_height which let us to give
an explicit window height. If set, automatic resize is enabled and the
height will be always the value of g:go_quickfix_height.

Previous case:

screen shot 2015-11-16 at 1 38 36 am

Improved case (auto height feature):

screen shot 2015-11-16 at 1 39 07 am

Closes #592

Right now we are just calling `cwindow` without any arguments. By
default Vim opens a view with an height of 10. But this doesn't look
good if you have only one single error. It also takes display estate for
small screens.

This PR introduces a new feature, which automatically resizes the
quickfix error for the number of errors. However to prevent consuming
all the view, it's capped at 10 lines (the default Vim value).

We also have a new setting, `g:go_quickfix_height` which let us to give
an explicit window height. If set, automatic resize is enabled and the
height will be always the value of `g:go_quickfix_height`.

Closes #592
fatih added a commit that referenced this pull request Nov 16, 2015
util: add auto quickfix window height for errors [GH-592]
@fatih fatih merged commit 6384e3a into master Nov 16, 2015
@fatih fatih deleted the auto-height-quickfix branch November 16, 2015 09:14
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.

Open quickfix with a smaller height if the number of items are less than 10
1 participant