bug#70637: closed (:box vertical bar artifacts at 'display boundaries)

[GNU bug Tracking System]

Your message dated Sat, 18 May 2024 12:17:44 +0300
with message-id <86o793eax3.fsf@gnu.org>
and subject line Re: bug#70637: :box vertical bar artifacts at 'display 
boundaries
has caused the debbugs.gnu.org bug report #70637,
regarding :box vertical bar artifacts at 'display boundaries
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
70637: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=70637
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> :box vertical bar artifacts at 'display boundaries
>
>
>
>
> Date: 
>
> Sun, 28 Apr 2024 20:50:31 -0400
>
>
>
>
> I have identified a display bug related to placing :box face parameters within 
> 'display strings.  Disable font-lock mode in *scratch* and evaluate:
>
> (insert "\n"
>  (propertize " " 'display (propertize "   " 'face '(:box t)))
>  (propertize "middle" 'face '(:box t))
>  (propertize " " 'display (propertize "   " 'face '(:box t))))
>
> So far so good: the box is correctly merged across all three elements and wraps 
> around them.  But now move point within and across the displayed boxed text.  
> Internal vertical divisions bars separating the 'display and normal :box 
> regions appear.  
>
> If you add a face box property to the blank flanking strings as well, this 
> prevents the internal boundary from appearing. 
>
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#70637: :box vertical bar artifacts at 'display boundaries
>
>
>
>
> Date: 
>
> Sat, 18 May 2024 12:17:44 +0300
>
>
>
>
> > From: JD Smith <jdtsmith@gmail.com>
> > Date: Sat, 11 May 2024 13:16:54 -0400
> > Cc: 70637@debbugs.gnu.org
> > 
> > > On May 11, 2024, at 5:41 AM, Eli Zaretskii <eliz@gnu.org> wrote:
> > > 
> > >> From: JD Smith <jdtsmith@gmail.com>
> > >> Date: Thu, 9 May 2024 09:31:28 -0400
> > >> Cc: 70637@debbugs.gnu.org
> > >> 
> > >> I presume this is a more general issue than just :box.  One idea is to add 
> > >> a warning to the Elisp section "Display
> > >> Specs That Replace The Text", perhaps at the end:
> > >> 
> > >> Note: certain `face' attributes such as `:box' can lead to display 
> > >> artifacts when applied to the replacing
> > >> text in a `display' specification.  These attributes may be incorrectly 
> > >> merged with adjacent non-`display'
> > >> `face' properties.  This can be mitigated by applying the `face' 
> > >> attributes directly to the text being
> > >> replaced, rather than (or in addition to) the `display' replacement text 
> > >> itself.
> > > 
> > > Actually, this is specific to :box, since only for that attribute we
> > > need to determine the beginning and the end of the box.
> > > 
> > >> Maybe a bit too wordy.
> > > 
> > > Yes, I agree.  If you can reword it to be specific to :box and to
> > > include an example, I think it would be good.
> > 
> > 
> > Feel free to edit:
>
> Thanks, I installed a variant of this, and I'm therefore closing this
> bug.
>
>
> --- End Message ---