Re: [O] [PATCH] Add a different prefix for past deadlines in the agenda view

[Sebastien Vauban]

Bastien,

Bastien wrote:
> "Sebastien Vauban" writes:
>
>> Though, one question: why did you change the spacing (and, in fact, the 
>> width)
>> of the leaders?
>>
>> -(defcustom org-agenda-scheduled-leaders '("Scheduled: " "Sched.%2dx: ")
>> +(defcustom org-agenda-scheduled-leaders '(" Scheduled: " "Sched.%3dx: ")
>>    "Text preceding scheduled items in the agenda view.
>>
>> (...)
>>
>> -(defcustom org-agenda-deadline-leaders '("Deadline:  " "In %3d d.: ")
>> +(defcustom org-agenda-deadline-leaders '("  Deadline: " " In %3d d.: " "%3d 
>> d. ago: ")
>
> For the change in "Scheduled:" and "Deadline:" it's because I think it's
> cleaner to align them on the right, like the "Sched." and "In %3d.:"
> strings.

I don't mind, I customize them anyway ;-)

> For the change in the width (from 11 to 12), it's because I wanted to
> correctly display "%3d d. ago: ", not %2d d. ago: ", so that now any string
> with a warning delay more than 100 will be correctly displayed. Before, we
> allowed 100+ days for the future deadlines, but not for past scheduled items
> (i.e. they were not aligned properly.)
>
> I feel the change is not too intrusive.

I think the change of width can impact people who customize a bit the look of
their agenda, and rely upong *all* the leaders to be of width 11. That was the
motivation for my other patch, about clocking info (to make it go into a
11-char slot).

Let's see how people react to that -- or if some are impacted.

Best regards,
  Seb

-- 
Sebastien Vauban