GNU bug report logs - #65896
30.0.50; folding text with text properties prevents background from extending past the newline

Previous Next

Package: emacs;

Reported by: StrawberryTea <look <at> strawberrytea.xyz>

Date: Tue, 12 Sep 2023 18:02:01 UTC

Severity: wishlist

Found in version 30.0.50

Done: Ihor Radchenko <yantar92 <at> posteo.net>

Bug is archived. No further changes may be made.

Full log


Message #46 received at 65896 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ihor Radchenko <yantar92 <at> posteo.net>
Cc: 65896 <at> debbugs.gnu.org, kevin.legouguec <at> gmail.com, look <at> strawberrytea.xyz
Subject: Re: bug#65896: 30.0.50; folding text with text properties prevents
 background from extending past the newline
Date: Fri, 22 Sep 2023 14:56:49 +0300
> From: Ihor Radchenko <yantar92 <at> posteo.net>
> Cc: look <at> strawberrytea.xyz, 65896 <at> debbugs.gnu.org, kevin.legouguec <at> gmail.com
> Date: Fri, 22 Sep 2023 10:12:57 +0000
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> >> Are you sure? I am looking at `extend_face_to_end_of_line' and it looks
> >> like there is nothing preventing it from accessing it->glyph_row->glyphs
> >> array to look backwards into preceding glyphs.
> >
> > First, glyphs have only partial information about the original face
> > properties.  IOW, the produced glyph row doesn't have all the info
> > that the original text had.
> >
> > And second, I don't see how the accessibility to the glyphs is
> > relevant to the issue at hand.  Suppose the visible text of a line is:
> >
> >    aaabbbbxxx
> >
> > where each letter represents some character with different face -- you
> > want Emacs to extend face 'a' after the end of the line, not the face
> > 'x', just because the line started with characters whose face was
> > 'a'?? that's what the OP said, AFAIU.
> 
> I am also not a big fan of using the first glyph/letter in the line.
> 
> What about not extending the face unless the last 2 glyphs have eq face
> or we have a single glyph in a row?

That's a backward-incompatible change, so we cannot do that.




This bug report was last modified 1 year and 175 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.