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

Avoid nvim_win_get_position for window position to support multigrid ui #3150

Merged

Conversation

akiyosi
Copy link
Contributor

@akiyosi akiyosi commented Jun 19, 2021

The current implementation of the scrollbar on the tooltip that coc.nvim pops up is not displayed in the correct position in the Neovim GUI that supports multigrid ui. See: akiyosi/goneovim#231
This fix resolves this issue.

The following is related information on this issue.

@chemzqm chemzqm changed the title fix float window position control to consider multigrid ui Avoid nvim_win_get_position for window position to support multigrid ui Jun 21, 2021
@chemzqm chemzqm merged commit 837941f into neoclide:master Jun 21, 2021
chemzqm added a commit that referenced this pull request Jun 22, 2021
91c1114 fix(diagnostic): avoid change range of diagnostics
cad8c6b fix(workspace): fix range of getSelectedRange with line mode
18f81ab chore(picker): use await to avoid test fail
3efce54 fix(diagnostic): no more emptyRange check (#3147)
837941f fix float window position control to consider multigrid ui (#3150)
046505c test(ci): use node12
acd723d fix(completion): avoid unnecessary redraw for floating window (#3151)
10cfe59 docs: inconsistent suggest.triggerCompletionWait value (#3152)
dafed9d fix: remove polyfill of Promise.prototyp.finally (#3153)
493ea2e fix(diagnostic): check diagnostic next to eol character
31e76d4 refactor(handler): fetch codelens for all buffers on CursorHold
e47e472 fix(diagnostic): adjust diagnostic range which pass last line
ba9c5c8 chore(yarn): upgrade esbuild.js
9b60af3 chore(bin): remove unnecessary check.js
179ca72 fix: typos found by github.com/crate-ci/typos (#3146)
70a7903 LSP 3.16.0 (#2756)
8cb6bdc fix(float): should give foldcolumn a default value (#3140)
171ff2b chore(bin): remove unnecessary check.js
1377f3b fix: yarn.lock (#3136)
@akiyosi akiyosi deleted the fix-floatwinScrollbarPos-in-multigrid branch August 6, 2022 11:31
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.

3 participants