Re: [O] bug#11249: 24.1.50; Overlay with face property causes calendar buffer to scroll

[Matt Lundin]

Matt Lundin <address@hidden> writes:

> Bastien <address@hidden> writes:
>
>> Toby Cubitt <address@hidden> writes:
>>
>>> In the patch I posted, I also took the opportunity to set
>>> `cursor-type' to nil when opening the calendar in `org-read-date'.
>>> The cursor obscures one digit of the selected date, making that bit
>>> harder to read (as well as looking ugly). Did you intend to reject
>>> this change too?
>>
>> I did because I found the not-bold-anymore face wasn't visible enough,
>> and the shallow cursor made it visible.
>>
>> I use inverse-video now for this face, so cursor-type nil is okay, I
>> applied a patch with this.
>
> This patch broke org-read-date. It will no longer parse date strings
> such as "Aug 15" correctly, instead inserting the current day.
>
[snip]
> If I comment out the following line:
>
>         (org-eval-in-calendar '(setq cursor-type nil))
>
> org-read-date works correctly.

Note: It seems that adding the optional argument KEEPDATE is necessary
for parsing to work. With the following line instead of the one above
org-read-date works as expected:

          (org-eval-in-calendar '(setq cursor-type nil) t)

Best,
Matt