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

Add pum_getpos() to return the position and size of the popup menu #4827

Closed
wants to merge 1 commit into from

Conversation

puremourning
Copy link
Contributor

Replaces #4823

Add pum_getpos() returning the same size and position information as CompleteChanged.

useful for plugins to avoid overlapping the PUM when placing popup windows. Also useful for testing.

@puremourning puremourning force-pushed the pum_getpos branch 4 times, most recently from f8d0570 to 5ca797c Compare August 17, 2019 16:49
@codecov-io
Copy link

Codecov Report

Merging #4827 into master will increase coverage by 0.07%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4827      +/-   ##
==========================================
+ Coverage    81.5%   81.57%   +0.07%     
==========================================
  Files         119      119              
  Lines      145708   145737      +29     
==========================================
+ Hits       118761   118891     +130     
+ Misses      26947    26846     -101
Impacted Files Coverage Δ
src/evalfunc.c 89.35% <100%> (+0.1%) ⬆️
src/version.c 92.1% <0%> (ø) ⬆️
src/ex_docmd.c 81.72% <0%> (+0.02%) ⬆️
src/channel.c 83.97% <0%> (+0.03%) ⬆️
src/edit.c 85.44% <0%> (+0.03%) ⬆️
src/terminal.c 81.35% <0%> (+0.03%) ⬆️
src/gui.c 63.75% <0%> (+0.05%) ⬆️
src/highlight.c 80.36% <0%> (+0.05%) ⬆️
src/undo.c 84.6% <0%> (+0.06%) ⬆️
src/misc2.c 87.14% <0%> (+0.07%) ⬆️
... and 27 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update df70775...c1d5fed. Read the comment docs.

@brammool brammool closed this in e9bd572 Aug 17, 2019
@puremourning puremourning deleted the pum_getpos branch August 17, 2019 17:52
manuelschiller pushed a commit to manuelschiller/vim that referenced this pull request Nov 10, 2019
Problem:    Cannot get size and position of the popup menu.
Solution:   Add pum_getpos(). (Ben Jackson, closes vim#4827)
sethfowler added a commit to sethfowler/neovim that referenced this pull request Dec 15, 2019
This is a straightforward port to neovim of the pum_getpos() API added
to vim in vim/vim#4827.
janlazo added a commit to janlazo/neovim that referenced this pull request Dec 29, 2019
Problem:    Cannot get size and position of the popup menu.
Solution:   Add pum_getpos(). (Ben Jackson, closes vim/vim#4827)
vim/vim@e9bd572

neovim#11562 backported the vim patch.
This patch only updates the runtime/doc/ files to match Vim.
janlazo added a commit to janlazo/neovim that referenced this pull request Dec 29, 2019
Problem:    Cannot get size and position of the popup menu.
Solution:   Add pum_getpos(). (Ben Jackson, closes vim/vim#4827)
vim/vim@e9bd572

neovim#11562 backported the vim patch.
This patch only updates the runtime/doc/ files to match Vim.
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.

None yet

2 participants