[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: specify time of day for org-resolve-clocks, not number of minutes
From: |
Bastien |
Subject: |
Re: specify time of day for org-resolve-clocks, not number of minutes |
Date: |
Thu, 13 Feb 2020 08:16:18 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) |
Hi Kyle and Dan,
Kyle Meyer <address@hidden> writes:
> Thanks, though sadly Dan had already taken the time to follow up with a
> patch:
>
> https://lists.gnu.org/archive/html/emacs-orgmode/2020-01/msg00175.html
Err, my bad, sorry Dan -- and thanks Kyle for the warning.
(I too hastily assume 1 thread = 1 topic, I should have checked.)
I reverted my changes and pushed Dan's commit to master.
I took the liberty of inlining the function and making the message a
bit more explicit. Dan, let me know if that's okay.
Interestingly, our (different and complementary) implementations may
lead to a new idea: your implementation is like the `k' option while
mine is like the `g' option (when you "got back"). I guess both can
make sense, and what the user expect is to be able to enter a number
of minutes *or* a HH:MM time spec in both `t' and `g'.
That would also have the advantage of having less options while still
having the possibility to use HH:MM. (Also, using org-read-date here
seems a bit too much here, but maybe that's okay.)
Dan, what do you think? Would you like to try implementing this or
can I give it a try?
Thanks,
--
Bastien