Thank you for your comments, Greg.
On 20/05/2021 01:06, Greg Minshall wrote:
>
> patches to patches... :) i think these are really just typos, rather
> than any useful substantial comment.
>
> - s/is enable (the default) or not:/is enabled (the default) or not:/
The patch removed this phrase.
> - i would suggest separate tables for >= 9.5 and < 9.5. just so the
> differences between with/without =electric-indent-mode= are easier to
> spot.
I have no idea how to express it in a better way. The tables are same
for Org >= 9.5 and for Org < 9.5 (modulo bugs in indentation fixed in 9.4.6).
Only default configuration evolves in time.
1. Before 9.4. (org-adapt-indentation t) electric-indent-mode disabled.
Bugs remained unnoticed. Users were unaware that default configuration
assumed indented text aligned to headline title.
2. 9.4. (org-adapt-indentation t) electric-indent-mode enabled.
Hot discussions and recommendations to disable electric-indent-mode
since even with (org-adapt-indentation headline-data) behavior was strange.
3. Recent changes. Bugs fixed, (org-adapt-indentation nil), so as earlier
text is not indented by default. Now absence of indentation is consistent
with settings.
> - s/C-j anywhere/C-j elsewhere/ (and, iiuc, maybe also the "unadorned"
> [C-j] table entry should probably be [C-j elsewhere]?)
>
> - s/lost formatting/lose formatting/
>
> - s/and, that is/and,/
Agree.
> - s/always indent relatively/always indent relative/
Not being a native speaker, I would either leave it as is or would change
all 3 phrases attempting to make their style more uniform.
a. "do not indent anything",
"always indent relatively to the element above",
"indent just ~PROPERTIES/LOGBOOK~ drawers relatively
to the current heading level"
b. "no special indentation",
"indentation of all elements is relative to the element above",
"only ~PROPERTIES/LOGBOOK~ drawers get indentation relative
to the current heading level"
If you think that your variant is better, I can add it to the patch.
> - s/the the element/to the element/
Agree.
Intermediate patch is attached.
>From 1881aacf22fda650408434394a96f3aaea2c6928 Mon Sep 17 00:00:00 2001
From: Max Nikulin
Date: Thu, 20 May 2021 22:48:58 +0700
Subject: [PATCH] org-faq.org: Fix typos and style in "setup for indenting"
* org-faq.org (What is the best setup for indenting?): Minor
corrections suggested by Greg Minshall.
---
org-faq.org | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/org-faq.org b/org-faq.org
index 0f8934af..54d5e403 100644
--- a/org-faq.org
+++ b/org-faq.org
@@ -1039,7 +1039,7 @@ In more details, ~org-adapt-indentation~ controls indentation with
regards to previous element. Apparent effect is increased indentation
for content of deeper nested headings. The variable accepts three
values: =nil= (no special indentation), =t= (always indent relatively
-the the element above) and =headline-data= (only indent the
+to the element above) and =headline-data= (only indent the
~PROPERTIES/LOGBOOK~ drawers relatively to the current level). Value
=t= is suitable for short entries especially if you plan to share your
documents with someone who does not use Emacs. If you just want to
@@ -1059,8 +1059,8 @@ With ~electric-indent-mode~ enabled:
|---+++|
| RET after a headline | Do not indent | Indent | Do not indent |
| C-j after a headline | Do not indent | Do not indent | Do not indent |
-| RET anywhere | Indent wrt previous| Indent wrt previous| Indent wrt previous|
-| C-j anywhere | Do not indent wrt prev | Do not indent wrt previous | Do not indent wrt previous |
+| RET elsewhere | Indent wrt previous| Indent wrt previous| Indent wrt previous|
+| C-j elsewhere | Do not indent wrt prev | Do not indent wrt previous | Do not indent wrt previous |
| Insert PROPERTIES/LOGBOOK | Do not indent | Indent wrt headline| Indent wrt headline|
With ~electric-indent-mode~ disabled:
@@ -1069,21 +1069,21 @@ With ~electric-indent-mode~ disabled:
|---+-++|
| RET after a headline | Do not indent | Do not indent | Do not indent |
| C-j after a headline | Do not indent | Indent | Do not indent |
-| RET | Do not indent | Do not indent wrt previous | Do not indent wrt previous |
-| C-j | Indent wrt previous | Indent wrt previous| Indent wrt previous|
+| RET elsewhere | Do not indent | Do not indent wrt previous | Do not
