unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Blyte Scholar <bs@blyte.net>
To: 72739@debbugs.gnu.org
Subject: bug#72739: [PATCH] * lisp/gnus/gnus-sum.el: Handle leafs with children in summary line
Date: Tue, 20 Aug 2024 17:44:48 -0400	[thread overview]
Message-ID: <87plq2c1in.fsf@rhipidon.mail-host-address-is-not-set> (raw)

[-- Attachment #1: Type: text/plain, Size: 735 bytes --]

Tags: patch

Tags: patch

Tags: patch


This patch adds customization options which handle the cases where a
thread leaf has both siblings and children. This allows using box
drawing characters to seamlessly connect all messages in a
thread. Previously, there would be messages could either be connected to
a sibling or a child, but not both.



In GNU Emacs 31.0.50 (build 2, x86_64-pc-linux-gnu, GTK+ Version
 3.24.43, cairo version 1.18.0) of 2024-08-20 built on rhipidon
Repository revision: 5d435afc0e389e107fcee8d63405078341592f97
Repository branch: master
System Description: Fedora Linux 40 (Workstation Edition)

Configured using:
 'configure --with-imagemagick --with-tree-sitter
 --with-native-compilation=aot --with-pgtk'


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-lisp-gnus-gnus-sum.el-Handle-leafs-with-children-in-.patch --]
[-- Type: text/patch, Size: 2595 bytes --]

From fca49a37ec7cc0c512290de93dd8311ca5a012d6 Mon Sep 17 00:00:00 2001
From: Blyte Scholar <bs@blyte.net>
Date: Tue, 20 Aug 2024 17:32:10 -0400
Subject: [PATCH] * lisp/gnus/gnus-sum.el: Handle leafs with children in
 summary line

(gnus-sum-thread-tree-leaf-with-other-root): New variable.
(gnus-sum-thread-tree-single-leaf-root): New variable.
(gnus-summary-prepare-threads):
Use gnus-sum-thread-tree-leaf-with-other-root and
gnus-sum-thread-single-leaf-root strings in summary line when they are
available.
---
 lisp/gnus/gnus-sum.el | 24 ++++++++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

diff --git a/lisp/gnus/gnus-sum.el b/lisp/gnus/gnus-sum.el
index a9caa83b15c..2fd342f3a0c 100644
--- a/lisp/gnus/gnus-sum.el
+++ b/lisp/gnus/gnus-sum.el
@@ -5193,12 +5193,26 @@ gnus-sum-thread-tree-leaf-with-other
   :type 'string
   :group 'gnus-thread)
 
+(defcustom gnus-sum-thread-tree-leaf-with-other-root nil
+  "With %B spec, used for a leaf with brothers and children.
+If nil use `gnus-sum-thread-tree-leaf-with-other' instead."
+  :version "31.1"
+  :type 'string
+  :group 'gnus-thread)
+
 (defcustom gnus-sum-thread-tree-single-leaf "\\-> "
   "With %B spec, used for a leaf without brothers."
   :version "22.1"
   :type 'string
   :group 'gnus-thread)
 
+(defcustom gnus-sum-thread-tree-single-leaf-root nil
+  "With %B spec, used for a leaf without brothers but children.
+If nil use `gnus-sum-thread-tree-leaf-single-leaf' instead."
+  :version "31.1"
+  :type 'string
+  :group 'gnus-thread)
+
 (defcustom gnus-summary-display-while-building nil
   "If non-nil, show and update the summary buffer as it's being built.
 If the value is t, update the buffer after every line is inserted.  If
@@ -5474,8 +5488,14 @@ gnus-summary-prepare-threads
 					  gnus-sum-thread-tree-indent))
 				      (cdr (reverse tree-stack))))
 		       (if (nth 1 thread)
-			   gnus-sum-thread-tree-leaf-with-other
-			 gnus-sum-thread-tree-single-leaf)))))
+                           (if (and gnus-sum-thread-tree-leaf-with-other-root
+                                    (cadar thread))
+                               gnus-sum-thread-tree-leaf-with-other-root
+			     gnus-sum-thread-tree-leaf-with-other)
+                         (if (and gnus-sum-thread-tree-single-leaf-root
+                                  (cadar thread))
+                             gnus-sum-thread-tree-single-leaf-root
+			   gnus-sum-thread-tree-single-leaf))))))
 	    (when (string= gnus-tmp-name "")
 	      (setq gnus-tmp-name gnus-tmp-from))
 	    (unless (numberp gnus-tmp-lines)
-- 
2.46.0


             reply	other threads:[~2024-08-20 21:44 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-20 21:44 Blyte Scholar [this message]
2024-08-24  9:14 ` bug#72739: [PATCH] * lisp/gnus/gnus-sum.el: Handle leafs with children in summary line Eli Zaretskii
2024-09-07  7:13   ` Eli Zaretskii
2024-08-31  8:14 ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87plq2c1in.fsf@rhipidon.mail-host-address-is-not-set \
    --to=bs@blyte.net \
    --cc=72739@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).