emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: :extend t inheritance


From: Ergus
Subject: Re: :extend t inheritance
Date: Sun, 27 Oct 2019 12:01:04 +0100
User-agent: K-9 Mail for Android

In GUI. The issue was in GUI. What I said about tui was that I didn't observe the issue in tui.

On October 27, 2019 6:51:07 AM GMT+01:00, Eli Zaretskii <address@hidden> wrote:
Date: Sat, 26 Oct 2019 23:13:40 +0000 (UTC)
From: Ergus <address@hidden>
Cc: address@hidden, address@hidden

Ohh right, that's the other commit. When fixing this issue I found another error I didn't see in tui. After EOL we
need to fill with the merged filtered face. To do that we need to add a stretch glyph; but we were just adding a
normal glyph with a ' ' so the face was not really extended, just the background color because the X display
engine always do that. But, for example, underline was not extended. This commit fixes just that.
stretch_ascent was moved out of the if(column_indicator) because the new stretch_glyphs needs it any way
and the two comments were 1) for old moved code not there anymore and 2) to describe the situation of the
glyphs color extension (mention in previous paragraph) that we don't need to work around anymore because
the glyph fills the entire screen now..

I'm still a bit confused: was the problem you found happening in GUI
frames or in text-mode frames? You say "tui", but then talk about
something X does...

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
reply via email to

[Prev in Thread] Current Thread [Next in Thread]