Hi, when selecting tags using fast selection and then de-selecting, the face doesn't toggle back to regular face. Fast selection means `org-use-fast-tag-selection' is t. Checked with emacs -Q + (setq org-use-fast-tag-selection t). Org 9.5, commit cc2490a7061955395c4f5a1a23a088044554a2f7. Emacs 29.0.50, hash 9f041cdfaca0f3def74eb7d3348bbf35c12bcc5b. -- Daniel Fleischer
[-- Attachment #1: Type: text/plain, Size: 241 bytes --] Daniel Fleischer <danflscr@gmail.com> writes: > Hi, when selecting tags using fast selection and then de-selecting, the > face doesn't toggle back to regular face. > > Fast selection means `org-use-fast-tag-selection' is t. See attached. [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: Patch --] [-- Type: text/x-patch, Size: 1005 bytes --] From 0352757d7012dc543bae4039deaed7bbd401bb9f Mon Sep 17 00:00:00 2001 From: Daniel Fleischer <danflscr@gmail.com> Date: Fri, 8 Oct 2021 12:55:04 +0300 Subject: [PATCH] lisp/org.el: fast tag selection disabled tag face A tag can be either selected in "current", "inherited" or disabled. When disabled, it should have the 'org-default face. Previously, if a tag was already selected, upon deselection it will still have an 'org-todo face. --- lisp/org.el | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lisp/org.el b/lisp/org.el index 18cb53af6..e6a0350c1 100644 --- a/lisp/org.el +++ b/lisp/org.el @@ -12440,8 +12440,7 @@ (defun org-fast-tag-selection (current inherited table &optional todo-table) (cond ((member tag current) c-face) ((member tag inherited) i-face) - (t (get-text-property (match-beginning 1) ' - face)))))))) + (t 'org-default))))))) (goto-char (point-min))))) (delete-overlay org-tags-overlay) (if rtn -- 2.33.0 [-- Attachment #3: Type: text/plain, Size: 23 bytes --] -- Daniel Fleischer
[-- Attachment #1: Type: text/plain, Size: 240 bytes --] > Daniel Fleischer <danflscr@gmail.com> writes: > >> Hi, when selecting tags using fast selection and then de-selecting, the >> face doesn't toggle back to regular face. >> It's better to have a default face instead of 'org-default face. [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: 0001-lisp-org.el-fast-tag-selection-disabled-tag-face.patch --] [-- Type: text/x-patch, Size: 1001 bytes --] From f269d3ad99b8ad570c19961831f38653046f07fb Mon Sep 17 00:00:00 2001 From: Daniel Fleischer <danflscr@gmail.com> Date: Fri, 8 Oct 2021 12:55:04 +0300 Subject: [PATCH] lisp/org.el: fast tag selection disabled tag face A tag can be either selected in "current", "inherited" or disabled. When disabled, it should have the 'org-default face. Previously, if a tag was already selected, upon deselection it will still have an 'org-todo face. --- lisp/org.el | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lisp/org.el b/lisp/org.el index 18cb53af6..1dfe70735 100644 --- a/lisp/org.el +++ b/lisp/org.el @@ -12440,8 +12440,7 @@ (defun org-fast-tag-selection (current inherited table &optional todo-table) (cond ((member tag current) c-face) ((member tag inherited) i-face) - (t (get-text-property (match-beginning 1) ' - face)))))))) + (t 'default))))))) (goto-char (point-min))))) (delete-overlay org-tags-overlay) (if rtn -- 2.33.0