purcell / ac-slime

Emacs auto-complete plugin for Slime symbols
109 stars 10 forks source link

Graphical glitches on ac-slime #4

Closed ghost closed 11 years ago

ghost commented 12 years ago

I am experiencing graphical glitches on the 'tooltip' of ac-slime, e.g. missing lines.

Here is a screenshot of such a glitch http://i42.tinypic.com/wvpufr.png

purcell commented 12 years ago

Interesting - I wonder if it's related to the ^M chars.

What versions of Emacs / slime / auto-complete.el do you have, and are you using any particular settings for 'slime-net-coding-system etc.?

m2ym commented 12 years ago

This might be caused by recent changes on popup.el. I also want to know about your environment.

ghost commented 12 years ago

I am using version 23.4 (latest stable on gnu site) on Win7-64.

And I finally got git working so you can look at my confiig here https://github.com/ClusterCat/.emacs.d

The problem might be the utf-8 encodings.

purcell commented 12 years ago

You have auto-complete 1.3.1; so perhaps it would be worth testing with the latest auto-complete code (https://github.com/m2ym/auto-complete) and popup.el (https://github.com/m2ym/popup-el)?

However, I also suspect the problem is the encodings. You're using 'utf-8 (not 'utf-8-unix) for your default coding system in init.el, but if you're using 'clojure-jack-in to start a swank back-end, then 'slime-net-coding-system will be 'utf-8-unix. So you could try explicitly setting your preferred coding systems to 'utf-8-unix to see if that helps.

Also, could you please copy and paste the result of the following command so that @m2ym and I can see the text that the popup should have contained? -

M-x pp-eval-expression RET (ac-slime-documentation "first") RET

Thanks!

-Steve

ghost commented 12 years ago

The output of eval is

"-------------------------\nclojure.core/first\n([coll])\n Returns the first item in the collection. Calls seq on its\n argument. If coll is nil, returns nil.\n"

I will try with the latest auto-complete and popup.el as soon as I have restored my config which is broken right now.

purcell commented 12 years ago

Hmm. That's odd, because the screenshot showed \r chars (^M), and they don't appear in the output you pasted... Very mysterious!

(BTW, if you ever want to compare configs, mine is at https://github.com/purcell/emacs.d -- the files of interest are init-slime.el and init-clojure.el.)

-Steve

ghost commented 12 years ago

Switching to latest github version of auto-complete and popup did not work. I also tried disabling everything not related so I had a basically uncostumized emacs with the same results.

Related: When evaluating the expression "(println (+ 2 3))" I get "5^M" in the slime repl

m2ym commented 12 years ago

Could you please try evaluate the following expression in \*scratch\* buffer?

(popup-menu* '(foo bar baz))

If you see the corrupted menu, that is the issue of popup.el (HEAD).

ghost commented 12 years ago

The menu displaying the three entries (foo bar baz) is not corrupted.

Bohtvaroh commented 12 years ago

I'm also getting some visual artifacts with latest git versions of auto-complete, ac-slime and popup:

http://imageshack.us/f/708/screenshot20120705at816.png/

Upd: fixed this by using http://www.emacswiki.org/emacs-en/PosTip#toc8

purcell commented 12 years ago

Hmm. If Pos Tip works with the same popup content, perhaps this means there is a bug in popup.el?

Bohtvaroh commented 12 years ago

Seems like that. If I understand correctly it's popup.el who's responsible for drawing these documentation visuals.

purcell commented 12 years ago

The popup author, @m2ym, commented above, but I don't think we've given him enough information to reproduce the problem. I certainly can't reproduce it locally; it seems to be specific to Windows -- I presume you're using that too, @Bohtvaroh?

Bohtvaroh commented 12 years ago

No, that's OS X. Aquamacs and latest popup from git. This graphical artifact isn't always reproduced, seems like it needs text to be wide enough. I can give some more information if it'd be relevant for the author.

purcell commented 11 years ago

Closing this -- there's no particular indication that this is specific to ac-slime, or that it's still an issue for users.