unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Kangas <stefankangas@gmail.com>
To: Durand <mmemmew@gmail.com>, emacs-devel@gnu.org
Subject: Re: [ELPA] Want to submit two packages "ilist" and "blist"
Date: Sun, 19 Sep 2021 03:15:42 -0700	[thread overview]
Message-ID: <CADwFkmm3494_q0OXLVTBkjG3w7wa6h5xCvkZZ4Wb7uMv8twAxA@mail.gmail.com> (raw)
In-Reply-To: <m2zgs9rxbp.fsf@gmail.com>

Durand <mmemmew@gmail.com> writes:

> I have written two Emacs pacakges, called "ilist" and "blist" (the
> former is the "engine", and hence a dependency, of the latter).  Now I
> am thinking about submitting the packages to GNU ELPA.
>
> The package "blist" is to display the list of bookmarks, in the sense of
> "bookmark.el", in a similar way as Ibuffer.

Thanks, this looks useful at a first glance, and something that I think
we should definitely welcome on GNU ELPA.

I wonder why you chose to write this as an entirely new package instead
of improving the bookmark list we already have in bookmark.el.  Could
you explain your rationale for this?

One thing that stands out is that you list bookmarks by category, like
in ibuffer, something that I don't think is currently possible with
`tabulated-list-mode'.

Some other scattered comments:

- Is any of this suitable for inclusion in bookmark.el?  I'm thinking of
  `blist-show-all-annotations', for example.

- I would add (defalias 'blist 'blist-list-bookmarks) for
  discoverability.

- blist-show-annotation says "No bookmarks to show" when it should
  probably say "No annotation for this bookmark".  I think?

- Instead of the in-buffer header, you could use `header-line-format',
  which means that the header stays on top even as you scroll down.



  reply	other threads:[~2021-09-19 10:15 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-19  4:41 [ELPA] Want to submit two packages "ilist" and "blist" Durand
2021-09-19 10:15 ` Stefan Kangas [this message]
2021-09-20  8:39   ` Durand
2021-09-19 12:45 ` Stefan Monnier
2021-09-19 16:33 ` Adam Porter
2021-09-20  0:10   ` Durand
2021-09-20  1:56     ` Adam Porter
2021-09-19 16:43 ` Adam Porter
2021-09-19 23:40   ` Durand
2021-09-19 17:01 ` [External] : " Drew Adams
2021-09-19 23:59   ` Durand
2021-09-20  4:18     ` Drew Adams
2021-09-20 11:19       ` Durand
2021-09-20 15:25         ` Drew Adams
2021-09-20 23:48   ` Richard Stallman
2021-09-21  0:45     ` Durand
2021-09-20 23:47 ` Richard Stallman
2021-09-21  0:39   ` Durand

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=CADwFkmm3494_q0OXLVTBkjG3w7wa6h5xCvkZZ4Wb7uMv8twAxA@mail.gmail.com \
    --to=stefankangas@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=mmemmew@gmail.com \
    /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).