unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Adam Porter <adam@alphapapa.net>
To: "Sebastián Monía" <sebastian@sebasmonia.com>
Cc: Jim Porter <jporterbugs@gmail.com>, Eli Zaretskii <eliz@gnu.org>,
	 emacs-devel <emacs-devel@gnu.org>
Subject: Re: [PATCH] Use vtable for eww-bookmarks
Date: Tue, 7 Jan 2025 06:59:58 -0600	[thread overview]
Message-ID: <CAO_b3FVxzGPM3yO+o-ggM8--WOSDGFFgPoL4PjOZ56_3PfdrMA@mail.gmail.com> (raw)
In-Reply-To: <36ff480e-0155-479d-87fb-9a85232dac47@app.fastmail.com>

[-- Attachment #1: Type: text/plain, Size: 1821 bytes --]

I don't think it's necessary to retire it. Your patch is an improvement,
no? If it's a step in the right direction, then why not proceed? Further
improvements can be made later.  AFAIK we're not talking about grand
features that would be a crime to make changes to later.  Most development
in Emacs doesn't happen with full consensus anyway, right? Usually someone
has an itch to scratch, and unless someone really objects or a procedure is
seriously violated, it gets scratched.  I've made some suggestions, but
they're only suggestions; they aren't conditions for merging.

Am I missing something here?  :)

On Tue, Jan 7, 2025, 06:23 Sebastián Monía <sebastian@sebasmonia.com> wrote:

> Hello everyone!
>
> I am ok to retire the patch, with no consensus on how to progress.
>
> Regards,
> Seb
>
> On Wed, Jan 1, 2025, at 11:25 PM, Sebastián Monía wrote:
> >
> > Adam Porter <adam@alphapapa.net> writes:
> > >  Also FWIW, I think the long-term goal ought to be to replace EWW's
> > >  bespoke bookmarks with Emacs bookmarks (i.e. any Emacs bookmark to an
> > >  HTTP(S) URL ought to be usable by EWW, and it ought to create such
> > >  bookmarks; EWW should be just another way of opening them).
> >
> > You can create Emacs bookmarks or EWW bookmarks. I use one or the other
> > for different cases. But that's more a peculiarity of how I work, than
> > proof that we need both, being honest.
> >
> > We could deprecate EWW-only bookmarks, I guess. If we plan to do that,
> > merging this patch makes no sense.
> > And then also drop the feature to re-arrange bookmarks, or make it work
> > in the context of standard bookmarks. Ditto for M-n and M-p to navigate
> > next/prev bookmark in EWW.
> >
> > --
> > Sebastián Monía
> > https://site.sebasmonia.com/
> >
> >
>

[-- Attachment #2: Type: text/html, Size: 2486 bytes --]

  reply	other threads:[~2025-01-07 12:59 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-05 22:53 [PATCH] Use vtable for eww-bookmarks Sebastián Monía
2024-11-06 12:21 ` Eli Zaretskii
2024-11-06 13:36   ` Sebastián Monía
2024-11-06 14:33     ` Eli Zaretskii
2024-11-06 14:43     ` Visuwesh
2024-11-06 16:52       ` Sebastián Monía
2024-11-06 20:49         ` Sebastián Monía
2024-11-07  2:00           ` Visuwesh
2024-11-07  2:00           ` Visuwesh
2024-11-20 19:27         ` Sebastián Monía
2024-11-11  7:38       ` Jim Porter
2024-11-23 19:12 ` Jim Porter
2024-11-27 20:02   ` Sebastián Monía
2024-11-28 19:49     ` Jim Porter
2024-12-01  5:33       ` Sebastián Monía
2024-12-01  6:39         ` Adam Porter
2024-12-01 16:17           ` Sebastián Monía
2024-12-01 19:31           ` Jim Porter
2024-12-01 22:12             ` Sebastián Monía
2024-12-09  4:52               ` Jim Porter
2024-12-10 19:16                 ` Sebastián Monía
2024-12-12  4:15                   ` Sebastián Monía
2024-12-12 15:25                     ` Sebastián Monía
2024-12-12 18:08                       ` Adam Porter
2024-12-12 19:37                         ` Sebastián Monía
2024-12-28 11:04                           ` Eli Zaretskii
2024-12-31  0:07                             ` Jim Porter
2024-12-31  5:43                               ` Sebastián Monía
2024-12-31 18:29                                 ` Adam Porter
2024-12-31 20:10                                   ` [External] : " Drew Adams
2025-01-02  4:25                                   ` Sebastián Monía
2025-01-07 12:23                                     ` Sebastián Monía
2025-01-07 12:59                                       ` Adam Porter [this message]
2025-01-07 18:55                                         ` Sebastián Monía
2025-01-07 19:27                                           ` Stefan Kangas
2025-01-07 19:41                                             ` Sebastián Monía
2025-01-07 21:01                                               ` Stefan Kangas
2025-01-07 22:47                                                 ` Jim Porter
2025-01-08  6:02                                                 ` Thierry Volpiatto
2025-01-08 15:52                                                   ` Adam Porter

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=CAO_b3FVxzGPM3yO+o-ggM8--WOSDGFFgPoL4PjOZ56_3PfdrMA@mail.gmail.com \
    --to=adam@alphapapa.net \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=jporterbugs@gmail.com \
    --cc=sebastian@sebasmonia.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).