Benjamin Haskell wrote:
> On Wed, 29 Sep 2010, Benjamin R. Haskell wrote:
>
> > This placement probably makes more sense (supersedes the prior patch),
> > but the ternaries are getting really ugly.
> >
> > Not sure what I was referring to in the first post (p_pvh didn't seem
> > to be used in prepare_tagpreview). There's somewhere though that
> > checks that a new window isn't taller than p_pvh (which is how I knew
> > the variable name was p_pvh).
> >
>
> Didn't see any response to this. It fixes the issue reported in two
> threads[1,2]. Updated to make it less ugly (no complicated ternaries),
> and match Vim coding style (I think -- didn't see a response to my query
> about that).
>
> [1] "Is this a bug with 'previewheight'?" - Daniel Vim - Sep 25
> http://groups.google.com/group/vim_use/browse_thread/thread/3c365eac4c8b814c
>
> [2] "how to set omnifunc complete window height (with img)" - Zac Lee - Sep 28
> http://groups.google.com/group/vim_use/browse_thread/thread/a2a99470e827662b
Is this the same as this todo entry:
Patch to use 'previewheight' for popup menu. (Benjamin Haskell, 2010 Sep 29)
Or another one?
--
Apologies for taking up the bandwidth with the apology. Anything else I
can apologise for ...... er no can't think of anything, sorry about that.
Andy Hunt (Member of British Olympic Apology Squad)
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ download, build and distribute -- http://www.A-A-P.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
