Kyle Meyer <k...@kyleam.com> writes:
Would you mind updating the patch to add a
test case along the lines of your ECM to
test-org-list/toggle-item?
Thanks.
Added the test in the attached patch.
Thanks, Kyle.
~ Nicholas Vollmer
>From 838a6a548396eecfa958161abb66f0a1719a9aef Mon Sep 17 00:00:00 2001
From: Nicholas Vollmer <iarchivedmywholel...@gmail.com>
Date: Fri, 31 Jul 2020 18:38:11 -0400
Subject: [PATCH] org-list: Operate on single line if no active region
* lisp/org-list.el (org-toggle-item): Operate on single line if no
active region.
---
lisp/org-list.el | 2 +-
testing/lisp/test-org-list.el | 7 +++++++
2 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/lisp/org-list.el b/lisp/org-list.el
index fb061b054..c43630d05 100644
--- a/lisp/org-list.el
+++ b/lisp/org-list.el
@@ -3032,7 +3032,7 @@ With a prefix argument ARG, change the region in a single item."
(if (org-region-active-p)
(setq beg (funcall skip-blanks (region-beginning))
end (copy-marker (region-end)))
- (setq beg (funcall skip-blanks (point-at-bol))
+ (setq beg (point-at-bol)
end (copy-marker (point-at-eol))))
;; Depending on the starting line, choose an action on the text
;; between BEG and END.
diff --git a/testing/lisp/test-org-list.el b/testing/lisp/test-org-list.el
index abd1a3c83..24a55464d 100644
--- a/testing/lisp/test-org-list.el
+++ b/testing/lisp/test-org-list.el
@@ -1109,6 +1109,13 @@ b. Item 2<point>"
(org-test-with-temp-text "* H\n:PROPERTIES:\n:A: 1\n:END:\n\n\nText"
(org-toggle-item nil)
(buffer-string))))
+ ;; When no region is marked and point is on a blank line
+ ;; only operate on current line
+ (should
+ (equal " \n* H :tag:"
+ (org-test-with-temp-text "<point> \n* H :tag:"
+ (org-toggle-item nil)
+ (buffer-string))))
;; When a region is marked and first line is a headline, all
;; headlines are turned into items.
(should
--
2.27.0
