Re: [emacs-wiki-discuss] Re: planner-appt or diary question

[Jim Ottaway]

>>>>> Seth Falcon <address@hidden> writes:

> On 16 Dec 2005, address@hidden wrote:
>> I would say that it could be merged in the forthcoming week.

> Ha!

Actually, I have been more forthcoming and merged it into
planner-appt, and Michael has merged it into planner, as you may have
seen. If it doesn't work properly it won't have any impact on the rest
of planner-appt.

> Did a bit of playing with the code.  Overall, seems to be working
> nicely.

> I notice that the code requires planner-id (haven't looked to see how
> this is being used).  At present, running this code turns on
> planner-id.  I haven't ever used planner-id and am not sure I want to
> unless I have to.  In testing, I am now seeing little {{Tasks:X}}
> markers in my task descriptions...

It doesn't actually need to require planner-id: originally I thought I
would use the task-id regexp defined there, but it's not precise
enough [covers all "{{<foo>:<bar>}}" constructs, not just
"{{Tasks:<bar>}}"], so I made my own.  I forgot that requiring
planner-id would install it as well, and I forgot to remove the
require.  Planner-appt doesn't require planner-id in that way, so the
merged version is fine in that regard.

> One request that goes for planner-appt in general: would it be
> possible to mark appts with no end time with just whitespace or '-'
> instead of "''''"?  There is a separate issue with the alignment as
> the following shows:

> [[2005.12.17]] |  12:00 | 13:00 | # Eat some food
> [[2005.12.20]] |  12:00 | ''''      | # Drink some coffee

> Wanted:

> [[2005.12.17]] |  12:00 | 13:00 | # Eat some food
> [[2005.12.20]] |  12:00 |   -   | # Drink some coffee

The "''''" was there because with something like 

[[2005.12.20]] |  12:00 |       | # Drink some coffee

the whitespace in the empty cell would disappear, leading to

[[2005.12.20]] |  12:00 || # Drink some coffee

which, if I remember correctly, would get interpreted as a separate
header field.  It appears that this doesn't happen any more; I will
remove the "''''" so that an empty cell is just whitespace.

What goes in an appt with no end time could be optional, but I think
that would require some modification of the schedule-based appointment
scanning code since the regexp to extract the time and task would have
to be modified to fit the option.  Is whitespace on its own acceptable?

> In the patch below, I renamed a few of the functions so that function
> names follow a consistent planner-appt-forthcoming-* pattern (and also
> match the documentation better).

Thank you.  I have changed these in planner-appt.  Can we switch to
that implementation in any future discussion? it would save confusion
and make patching easier.

> I also changed the formatting for appts on the same day so that you
> now get (paste into a planner-mode buffer to see proper alignment) The
> point was to use a bit less ink with " - " instead of "------":

Yes, that looks good.

What kind of ink do you use, is it expensive?! 

Thank you for the comments, and for the idea, and for the code: I think
it is a very useful addition to planner-appt.

Regards,

-- 
Jim Ottaway