htop background issues

[deadtrickster]

As requested, emacs -Q screenshot:

[deadtrickster]

emacs is the HEAD of master

[deadtrickster]

maybe related (git diff):

also interesting to note - emacs string face used. They should be white. Maybe this conflicts with escape codes. Ansi-term is ok.

[deb0ch]

One more info that would be useful: what version/commit of emacs-libvterm are you using?

[deadtrickster]

1b03e4e

[deadtrickster]

strings highlighting can be turned of by (setq-local font-lock-keywords-only t) in the mode hook.

[akermu]

@deadtrickster Is it possible to give me a short elisp configuration loaded with emacs -Q, which reproduces the issue?

[deadtrickster]

Yeah, will do on weekend.

[deadtrickster]

I can reproduce it with emacs -Q and two lines:

(add-to-list 'load-path "../emacs-libvterm") 
(require 'vterm)

[akermu]

I cannot reproduce this here, but I pushed a update regarding color handling in general. So can you reproduce this with the latest master? And which OS are you using, it might be related to the version of libvterm?

[deadtrickster]

I have latest ubuntu here.

htop inside xfce4-terminal:

htop inside emacs:

Same color schema. I would say black background handling is suspicious.

[deadtrickster]

to rule out htop-specific glitches I tried scripts from this page:
http://www.lihaoyi.com/post/BuildyourownCommandLinewithANSIescapecodes.html#background-colors

Black and magenta backgrounds seem to be off

[akermu]

With 920d847 the script prints now the correct colors. Does this fix your issue?

[deadtrickster]

I rebuilt emacs, libterm, now buffer background is always black, and still colors messed up in htop and scripts :-)

[deadtrickster]

Actually black background is ok, not magenta though:

Note how setting background to magenta cleared black background to default-bg and set magenta as foreground.

[akermu]

Magenta is fixed with 622cde3. The htop background issue seems to be a different issue.

[akermu]

@deadtrickster Is this still reproduceable with the latest master, since #82 is merged?

[deadtrickster]

nope, thanks a lot!