GNU bug report logs - #54095
Feature request: make 'outline-blank-line' defcustom and local-safe.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 54095 in the body.
You can then email your comments to 54095 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: Jeff Norden <jeff <at> jnorden.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Feature request: make 'outline-blank-line' defcustom and local-safe.
Date: Mon, 21 Feb 2022 14:12:40 -0600

[Message part 1 (text/plain, inline)]

I've been playing with outline-mode for the first time in a while. I've 
got a
simple suggestion: it seems like the `outline-blank-line' variable ought 
to be
customizable and file-local safe. It causes a single blank line to 
separate
hidden heading lines when there is at least one blank line in the 
un-hidden
text.

If you leave blank lines before some headings and not others, it 
provides an
additional visual way of grouping them, without the need to create 
another
heading level. This is handy if the distinction is temporary, as in: 
"I'm done
with these sections, but still working on those."

A trivial patch is below (and also attached). Version 22.1 is a guess on 
my
part, someone with better git skills than me could probably check it.

Thanks!
-Jeff

============================================================
--- outline.el.orig    2022-02-21 12:08:50.124838087 -0600
+++ outline.el    2022-02-21 13:05:59.087075549 -0600
@@ -294,8 +294,11 @@
 (defvar outline-mode-hook nil
   "This hook is run when outline mode starts.")

-(defvar outline-blank-line nil
-  "Non-nil means to leave unhidden blank line before heading.")
+(defcustom outline-blank-line nil
+  "Non-nil means to leave an unhidden blank line before headings."
+  :type 'boolean
+  :version "22.1")
+;;;###autoload(put 'outline-blank-line 'safe-local-variable 'booleanp)

 ;;;###autoload
 (define-derived-mode outline-mode text-mode "Outline"
============================================================

[outline_el.diff (text/x-diff, attachment)]

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Jeff Norden <jeff <at> jnorden.com>
Cc: 54095 <at> debbugs.gnu.org
Subject: Re: bug#54095: Feature request: make 'outline-blank-line' defcustom
 and local-safe.
Date: Tue, 22 Feb 2022 14:51:57 +0100

Jeff Norden <jeff <at> jnorden.com> writes:

> A trivial patch is below (and also attached). Version 22.1 is a guess
> on my part, someone with better git skills than me could probably
> check it.

Thanks; pushed to Emacs 29.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

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

From: Juri Linkov <juri <at> linkov.net>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 54095 <at> debbugs.gnu.org, Jeff Norden <jeff <at> jnorden.com>
Subject: Re: bug#54095: Feature request: make 'outline-blank-line' defcustom
 and local-safe.
Date: Tue, 22 Feb 2022 19:05:39 +0200

>> A trivial patch is below (and also attached). Version 22.1 is a guess
>> on my part, someone with better git skills than me could probably
>> check it.
>
> Thanks; pushed to Emacs 29.

Maybe the same way as outline-blank-line defines whether to show ^J
between outline headings, the same variable or another similar variable
could define whether to also include ^L as a separator between outlines.
This will fix the problem reported in bug#51016.

Then 'outline-blank-line' could be e.g. a regexp like [^J^L]+.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Juri Linkov <juri <at> linkov.net>
Cc: 54095 <at> debbugs.gnu.org, Jeff Norden <jeff <at> jnorden.com>
Subject: Re: bug#54095: Feature request: make 'outline-blank-line' defcustom
 and local-safe.
Date: Wed, 23 Feb 2022 13:16:08 +0100

Juri Linkov <juri <at> linkov.net> writes:

> Maybe the same way as outline-blank-line defines whether to show ^J
> between outline headings, the same variable or another similar variable
> could define whether to also include ^L as a separator between outlines.
> This will fix the problem reported in bug#51016.
>
> Then 'outline-blank-line' could be e.g. a regexp like [^J^L]+.

Hm...  I think a separate variable would be better for that -- having a
boolean here is nice for easy on/off in .dir-locals.el.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Previous Next

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