unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stephen Berman <stephen.berman@gmx.net>
To: Glenn Morris <rgm@gnu.org>
Cc: Stefan Monnier <monnier@iro.umontreal.ca>, emacs-devel@gnu.org
Subject: Re: New version of todo-mode.el (announcement + user guide)
Date: Wed, 12 Jun 2013 23:37:58 +0200	[thread overview]
Message-ID: <8761xjdn7t.fsf@rosalinde.fritz.box> (raw)
In-Reply-To: <6emwqvxmpo.fsf@fencepost.gnu.org> (Glenn Morris's message of "Wed, 12 Jun 2013 13:28:35 -0400")

On Wed, 12 Jun 2013 13:28:35 -0400 Glenn Morris <rgm@gnu.org> wrote:

> Stephen Berman wrote:
>
>> - If Glenn Morris approves, can I install the patch I included for
>>   diary-lib.el?
>
> Fine by me, though if possible I think it would be better to add some
> new diary hooks(s) so that you can do whatever it is you want to do
> without needing to put todo-specific stuff in diary-lib. But feel free.

Thanks for the go-ahead.  I do agree with you that a hook would be
better and in fact gave it some thought, but couldn't see how to do it
using an existing hook.  I didn't consider adding a new hook, and I'm
not sure I what that should be.  The problem is, after the button is
clicked in the Fancy Diary display, if the todo file is in a buffer, it
will probably be narrowed, so it has to be widened before going to the
position of the marker, and then it has to be narrowed again afterwards
to get the proper display.  So either there have to be two hooks
sandwiching the goto-char for the marker, or the latter has to be passed
to the hook function.  Neither really seems much better than the ad hoc
patch I posted, but it's not unlikely I'm overlooking a better
alternative.  If you have any ideas, I'm all ears.

>> - I also have a question about documentation.  The user guide I posted
>>   is certainly too long and detailed for the commentary section of the
>>   source code, and I guess also for the Emacs manual.
>
> It is longer than the entire current todo-mode.el is...

Yes, but then the new version is nearly seven times longer than the old
one.  Still, I'd welcome suggestions for slimming down the
documentation.  Maybe when others use the package some of the features
will be found to be dispensible.

> (244k will make it about the 15th largest Lisp file in Emacs.
> I doubt anyone has time to review that in detail.)

I don't expect that (though I'd welcome any feedback).  The package has
had an excessively long genesis (I'm not a fast coder) and I was
probably strategically unwise not to post earlier, less featureful,
versions.

Steve Berman



  parent reply	other threads:[~2013-06-12 21:37 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-09 23:31 New version of todo-mode.el (announcement + user guide) Stephen Berman
2013-06-10 13:24 ` Bastien
2013-06-10 14:35   ` Stephen Berman
2013-06-10 14:49     ` Bastien
2013-06-10 20:51       ` New version of todo-mode.el (code) Stephen Berman
2013-08-30 18:31         ` Jambunathan K
2013-09-08 21:09           ` Stephen Berman
2013-06-10 14:52     ` New version of todo-mode.el (announcement + user guide) Óscar Fuentes
2013-06-10 20:52       ` Stephen Berman
2013-06-11  0:20         ` Stefan Monnier
2013-06-11 18:36           ` Stephen Berman
2013-06-11 21:48             ` Stefan Monnier
2013-06-12 21:37               ` Stephen Berman
2013-06-13  1:06                 ` Stefan Monnier
2013-06-13 20:53                   ` Stephen Berman
2013-06-12 17:28             ` Glenn Morris
2013-06-12 21:26               ` Stefan Monnier
2013-06-12 21:37               ` Stephen Berman [this message]
2013-06-13  1:18                 ` Stefan Monnier
2013-06-13 20:53                   ` Stephen Berman
2013-06-14  0:21                     ` Stefan Monnier
2013-06-14 21:37                       ` Stephen Berman
2013-06-15  0:40                         ` Glenn Morris
2013-06-15  1:49                         ` Stefan Monnier
2013-06-15 12:52                           ` Stephen Berman
2013-06-16  0:44                             ` Stefan Monnier
2013-06-16 22:52                               ` Stephen Berman
2013-06-17  0:37                                 ` Stefan Monnier
2013-06-17 19:50                                 ` Glenn Morris
2013-06-17 22:33                                   ` Stephen Berman
2013-06-12 18:30             ` Wolfgang Jenkner
2013-06-12 21:38               ` Stephen Berman
2013-06-13  1:24                 ` Wolfgang Jenkner
2013-06-13 20:54                   ` Stephen Berman
2013-06-13 10:59 ` Vitalie Spinu
2013-06-13 20:54   ` Stephen Berman
2013-08-31  3:55 ` Jambunathan K

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.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=8761xjdn7t.fsf@rosalinde.fritz.box \
    --to=stephen.berman@gmx.net \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=rgm@gnu.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.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).