GNU bug report logs - #67713
[PATCH] 30.0.50; Small change to display.texi

Previous Next

Package: emacs;

Reported by: Arash Esbati <arash <at> gnu.org>

Date: Fri, 8 Dec 2023 16:04:02 UTC

Severity: normal

Tags: patch

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Arash Esbati <arash <at> gnu.org>
Subject: bug#67713: closed (Re: bug#67713: [PATCH] 30.0.50; Small change
 to display.texi)
Date: Sat, 09 Dec 2023 11:08:01 +0000
[Message part 1 (text/plain, inline)]
Your bug report

#67713: [PATCH] 30.0.50; Small change to display.texi

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 67713 <at> debbugs.gnu.org.

-- 
67713: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=67713
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Eli Zaretskii <eliz <at> gnu.org>
To: Arash Esbati <arash <at> gnu.org>
Cc: 67713-done <at> debbugs.gnu.org
Subject: Re: bug#67713: [PATCH] 30.0.50; Small change to display.texi
Date: Sat, 09 Dec 2023 13:06:26 +0200
> From: Arash Esbati <arash <at> gnu.org>
> Cc: 67713 <at> debbugs.gnu.org
> Date: Fri, 08 Dec 2023 17:43:09 +0100
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > Thanks, but why is it so important to follow the order in the @item?
> 
> One point (which is minor) is that my brain can parse that information
> easier if they are in order.  The major point is that couple of lines
> above in display.texi (and also in other places in the manual), the
> information is provided in that order:
> 
> @item @code{(:color @var{color} :style @var{style} :position @var{position})}
> @var{color} is either a string, or the symbol @code{foreground-color},
> meaning the foreground color of the face.  Omitting the attribute
> @code{:color} means to use the foreground color of the face.
> @var{style} should be a symbol @code{line} or @code{wave}, meaning to
> use a straight or wavy line.  Omitting the attribute @code{:style}
> means to use a straight line.  @var{position}, if non-@code{nil}, means to
> display the underline at the descent of the text, instead of at the
> baseline level.  If it is a number, then it specifies the amount of
> pixels above the descent to display the underline.
> 
> But I'm easy, feel free to reject; it is really a minor issue.

I installed this on the emacs-29 branch, and I'm closing this bug.

Thanks.

[Message part 3 (message/rfc822, inline)]
From: Arash Esbati <arash <at> gnu.org>
To: "emacs-bugs" <bug-gnu-emacs <at> gnu.org>
Subject: [PATCH] 30.0.50; Small change to display.texi
Date: Fri, 08 Dec 2023 17:03:03 +0100
Hi all,

I'd like to suggest a small change to display.texi where the text
follows the code in line 2725 (line break added for legibility):

  @item @code{(:line-width (@var{vwidth} . @var{hwidth})
               :color @var{color} :style @var{style})}

--8<---------------cut here---------------start------------->8---
diff --git a/doc/lispref/display.texi b/doc/lispref/display.texi
index 4dbb4afb20d..2056b8552da 100644
--- a/doc/lispref/display.texi
+++ b/doc/lispref/display.texi
@@ -2735,16 +2735,16 @@ Face Attributes
 single number @var{n} instead of a list, such case is equivalent to
 @code{((abs @var{n}) . @var{n})}.

+The value @var{color} specifies the color to draw with.  The default
+is the background color of the face for 3D boxes and
+@code{flat-button}, and the foreground color of the face for other
+boxes.
+
 The value @var{style} specifies whether to draw a 3D box.  If it is
 @code{released-button}, the box looks like a 3D button that is not
 being pressed.  If it is @code{pressed-button}, the box looks like a
 3D button that is being pressed.  If it is @code{nil},
 @code{flat-button} or omitted, a plain 2D box is used.
-
-The value @var{color} specifies the color to draw with.  The default
-is the background color of the face for 3D boxes and
-@code{flat-button}, and the foreground color of the face for other
-boxes.
 @end table

 @item :inverse-video
--8<---------------cut here---------------end--------------->8---

Best, Arash



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

Previous Next


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