Re: [PATCH] org-agenda.el: Complete multiple todo keywords

2020-05-01 Thread Kyle Meyer
Applied (d23bd9187).

Re: [PATCH] org-agenda.el: Complete multiple todo keywords

2020-05-01 Thread akater
Kyle Meyer writes: > Is akater the name you prefer > listed on that site and in the commit message? Yes. I will use git-am complaint patch formatting next time; thank you for the guidelines. signature.asc Description: PGP signature

Re: [PATCH] org-agenda.el: Complete multiple todo keywords

2020-04-30 Thread Kyle Meyer
akater writes: > Done. New patch is attached. Thanks for the update and the expanded explanations. The patch looks good to me. As a minor note for future patches, it's easier on the receiver if you include the non-patch commentary in one of the ways that git-am can automatically detect and

Re: [PATCH] org-agenda.el: Complete multiple todo keywords

2020-04-30 Thread akater
Kyle Meyer writes: > Thanks for the patch. Looks like a nice improvement to me. > > akater writes: > >> * lisp/org-agenda.el (org-todo-list): Use completing-read-multiple >> instead of completing-read when selecting todo keywords to filter by >> in Agenda. > > This and the rest of the lines

Re: [PATCH] org-agenda.el: Complete multiple todo keywords

2020-04-29 Thread Kyle Meyer
Thanks for the patch. Looks like a nice improvement to me. akater writes: > * lisp/org-agenda.el (org-todo-list): Use completing-read-multiple > instead of completing-read when selecting todo keywords to filter by > in Agenda. This and the rest of the lines were unwrapped. Could you wrap

[PATCH] org-agenda.el: Complete multiple todo keywords

2020-04-29 Thread akater
* lisp/org-agenda.el (org-todo-list): Use completing-read-multiple instead of completing-read when selecting todo keywords to filter by in Agenda. * lisp/org-agenda.el (org-todo-list): Fix a typo in the prompt. There is minor UX cost to Helm users: while candidates list used to appear