emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Matt Lundin <mdl@imapmail.org>
To: Bastien <bzg@altern.org>
Cc: OrgMode ML <emacs-orgmode@gnu.org>,
	Sebastien Vauban
	<public-wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@plane.gmane.org>
Subject: Re: [PATCH] Add a different prefix for past deadlines in the agenda view
Date: Sun, 24 Mar 2013 17:12:04 -0500	[thread overview]
Message-ID: <87y5dcla6j.fsf@fastmail.fm> (raw)
In-Reply-To: <8738wgg51w.fsf@bzg.ath.cx> (Bastien's message of "Thu, 28 Feb 2013 10:29:15 +0100")

Bastien <bzg@altern.org> writes:

> "Sebastien Vauban"
> <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org> writes:
>
>> 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).
>
> The thread about your other patch is still open, see my last reply.
>
> Also, a width of 12 should make it easier to fix alignment issues,
> so feel free to have a go and fix the ones that disturb you.  Please
> remember not every user has a 3k configuration file, so implementing
> TRT with emacs -Q is highly recommended here :)

I just upgraded org-mode for the first time in a while and found these
new default settings for deadline and scheduled leaders somewhat
disorienting. 

A 12-char width causes misalignment with a sample file and emacs -Q.
(See output below).

Here is an example of the new default behavior with emacs -Q

--8<---------------cut here---------------start------------->8---
Day-agenda (W12):
Sunday     24 March 2013
               8:00...... ----------------
  test:       10:00......  Scheduled:  TODO Scheduled at a time
  test:       10:00...... An appointment
              10:00...... ----------------
              12:00...... ----------------
              14:00...... ----------------
              16:00...... ----------------
              16:59...... now - - - - - - - - - - - - - - - - - - - - - - - - -
              18:00...... ----------------
              20:00...... ----------------
  test:       Sched.  3x:  TODO A scheduled item
  test:        Scheduled:  TODO Another scheduled item
  test:         3 d. ago:  TODO An overdue item
  test:        In   2 d.:  Deadline
                          ^ 
                          Note the extra space here
--8<---------------cut here---------------end--------------->8---

And here is an example of the old default behavior (11 char. spacing):

--8<---------------cut here---------------start------------->8---
Day-agenda (W12):
Sunday     24 March 2013
               8:00...... ----------------
  test:       10:00...... Scheduled:  TODO Scheduled at a time
  test:       10:00...... An appointment
              10:00...... ----------------
              12:00...... ----------------
              14:00...... ----------------
              16:00...... ----------------
              17:06...... now - - - - - - - - - - - - - - - - - - - - - - - - -
              18:00...... ----------------
              20:00...... ----------------
  test:       Sched. 3x:  TODO A scheduled item
  test:       Scheduled:  TODO Another scheduled item
  test:       In  -3 d.:  TODO An overdue item
  test:       In   2 d.:  Deadline
--8<---------------cut here---------------end--------------->8---

> 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.

FWIW, I find the old default much cleaner and easier to scan visually. I
know this can be customized, but might I suggest that we revert the
default to 11-characters and left aligned text?

Best,
Matt

P.S. Thanks, as always, Bastien for all the amazing work you've been
doing as maintainer!

  reply	other threads:[~2013-03-24 22:12 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-04 21:58 [PATCH] Add a different prefix for past deadlines in the agenda view Sebastien Vauban
2013-02-04 22:45 ` Bastien
2013-02-27 13:56   ` Bastien
2013-02-28  8:17     ` Sebastien Vauban
2013-02-28  8:29       ` Bastien
2013-02-28  9:21         ` Sebastien Vauban
2013-02-28  9:29           ` Bastien
2013-03-24 22:12             ` Matt Lundin [this message]
2013-03-25  5:57               ` Bastien
2013-02-28  8:30       ` Bastien
2013-02-28  9:42         ` Sebastien Vauban
2013-02-28  9:48         ` Andreas Leha
2013-02-28  9:55           ` Sebastien Vauban

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87y5dcla6j.fsf@fastmail.fm \
    --to=mdl@imapmail.org \
    --cc=bzg@altern.org \
    --cc=emacs-orgmode@gnu.org \
    --cc=public-wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@plane.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).