Renato Ferreira <renatof...@gmail.com> writes: > (org-todo): Fast selection should only be shown if a state argument was not > used > * list/org.el (org-todo): Respect argument when called from elisp > when calling from elisp (such as in `org-clock-in').
Thanks for the patch! I hit this earlier today but hadn't yet looked into it. Testing this out, it fixes the issue introduced by f1c030bed (Prefix argument to `org-todo' forces stage change logging, 2019-08-14), while keeping the intended C-u behavior change from that patch. > @@ -10121,7 +10118,9 @@ When called through ELisp, arg is also interpreted in > the following way: > (user-error "State `%s' not valid in this file" > arg)) > ((nth (1- (prefix-numeric-value arg)) > org-todo-keywords-1)))) > + ((and org-todo-key-trigger > org-use-fast-todo-selection) > + ;; Use fast selection. > + (org-fast-todo-selection this)) > ((null member) (or head (car org-todo-keywords-1))) > ((equal this final-done-word) nil) ;-> make empty > ((null tail) nil) ;-> first entry This patch didn't apply for me, and the hunk counts look off. It should be -10121,6, no? Anyway, I've adjusted it so that it applies and pushed it to master. (I made a typo fix and a few small tweaks to your commit message.) -- Kyle