unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Kangas <stefankangas@gmail.com>
To: "Tassilo Horn" <tsdh@gnu.org>, "Johann Klähn" <johann@jklaehn.de>
Cc: emacs-devel@gnu.org, Philip Kaludercic <philipk@posteo.net>
Subject: Re: [NonGNU ELPA] New package: eldoc-diffstat
Date: Sat, 14 Dec 2024 13:24:29 +0000	[thread overview]
Message-ID: <CADwFkmmDSPoOPMN6ywR=OyPjr6KX3nAHxs9jwyV49xi5oAhf=A@mail.gmail.com> (raw)
In-Reply-To: <87r06b72oe.fsf@gnu.org>

Tassilo Horn <tsdh@gnu.org> writes:

>> I would like to submit a new package to NonGNU ELPA: eldoc-diffstat.
>>
>>   https://github.com/kljohann/eldoc-diffstat/
>
> I'm using it since a couple of days using package-vc and like it very
> much, so I took the liberty to add it to NonGNU ELPA.

Thanks, this seems pretty useful.  I can see myself wanting to have it
on sometimes, but not always, so I'd appreciate it being a minor mode
that I could toggle, instead of just a function.

When the mode is toggled on, it could detect automatically that it's in
a magit buffer, and do the `eldoc-add-command` setup for users
automatically.

Should we have something like this built-in?

Here are some more wishlist items:

- An option to use a maximum (or fixed) number of lines
- Caching the results for the current buffer to get instant results

> I think you could add a bit more configuration hints to the README.md,
> e.g., where you say that one should call eldoc-diffstat-setup in the
> desired buffer or mode hook, provide some typical examples like:
>
>   (add-hook 'log-view-mode-hook #'eldoc-diffstat-setup)
>   (add-hook 'vc-annotate-mode-hook #'eldoc-diffstat-setup)
>   (add-hook 'magit-status-mode-hook #'eldoc-diffstat-setup)
>   (add-hook 'magit-log-mode-hook #'eldoc-diffstat-setup)

+1

There could also be a global minor mode that would enable this in as
many places as possible.



  reply	other threads:[~2024-12-14 13:24 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-09 20:31 [NonGNU ELPA] New package: eldoc-diffstat Johann Klähn
2024-12-13 20:23 ` Tassilo Horn
2024-12-14 13:24   ` Stefan Kangas [this message]
2024-12-14 10:11 ` Philip Kaludercic

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='CADwFkmmDSPoOPMN6ywR=OyPjr6KX3nAHxs9jwyV49xi5oAhf=A@mail.gmail.com' \
    --to=stefankangas@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=johann@jklaehn.de \
    --cc=philipk@posteo.net \
    --cc=tsdh@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).