company-mode / company-quickhelp

Documentation popup for Company
GNU General Public License v3.0
374 stars 33 forks source link

Quickhelp showing blank popups Emacs26.1 #90

Closed connorjsmith closed 6 years ago

connorjsmith commented 6 years ago

Relatively new to emacs debugging, so please let me know if there's anything I can do to help triage/debug this.

I'm using Emacs 26.1 on Ubuntu 16.04

blank tip

expez commented 6 years ago

Is there a doc for that function? That is, does this happen always or does it erroneously pop up when there is no doc to show?

srustamo commented 6 years ago

Looks like the same issue here. The docstring is there in my case, can be seen with C-c h while in completion session, setup as described here. Also note the weird size and position of the popup (marked with oval).

screenshot 2018-07-03 21 50 26
connorjsmith commented 6 years ago

Yeah the documentation is available via f1 and C-c h. Popup doesn't occur when there is no documentation available (as expected). Also what's weird is the size of the popup changes as if it is sizing for longer/shorter docstrings, just no text appears.

srustamo commented 6 years ago

Played a bit with straight 'emacs-jedi' setup (without 'company-mode'), get a bit better result (see screenshot), fwiw.

screenshot 2018-07-05 18 10 12
Kaali commented 6 years ago

The empty popup happens to me also. It fixes itself if I manually call (pos-tip-show "foo") once. Don't really know what the state problem is, but the bug is most probably in pos-tip itself.

Kaali commented 6 years ago

Seems to happen because pos-tip shows a temporary post-tip when calculating coordinates. Which happens after quickhelp shows it's pos-tip. When tracing pos-tip-show it's called twice when quickhelp activates, once with pos-tip-show and after that pos-tip-show-no-propertize. Which calls pos-tip-show with empty contents, and thus overrides the popup.

To disable this behavior you can set pos-tip-use-relative-coordinates to t. Which fixes the problem for me.

expez commented 6 years ago

Thanks for investigating and reporting back @Kaali! :+1: