dense-analysis / dense-analysis/ale

Remove/replace check for 'set_balloons' in ale#balloon#MessageForPos

Open
#2,697 2 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Vim Script
Stars
14k
Forks
1.5k
Avg merge
17h 49m
Merged PRs (30d)
1

Description

These days many projects support balloons which is awesome, but they are all overriding the `set balloonexpr=...`
for example:
`vim-go - go#tool#DescribeBalloon()`
`ale - ale#balloon#Expr()`
also I have my own custom balloon logic for previewing folds.

It would be nice if `ale#balloon#MessageForPos` can return messages without checking
`let g:ale_set_balloons = 1` because it will override the `set balloonexpr=...` (maybe some other variable can be introduced)

This way we'll be able to write our own balloonexpr func and choose when to show contents of folds, when linter warnings and when golang definitons.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.