GNU bug report logs - #75354
29.4; eww buffer is not displayed correctly when used from bookmark-jump

Previous Next

Package: emacs;

Reported by: Thierry Volpiatto <thievol <at> posteo.net>

Date: Sat, 4 Jan 2025 16:15:02 UTC

Severity: normal

Found in version 29.4

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Ship Mints <shipmints <at> gmail.com>
To: Thierry Volpiatto <thievol <at> posteo.net>
Cc: eliz <at> gnu.org, 75354 <at> debbugs.gnu.org
Subject: bug#75354: (29.4; eww buffer is not displayed correctly when used from bookmark-jump )
Date: Fri, 14 Mar 2025 06:50:02 -0400
[Message part 1 (text/plain, inline)]
On Fri, Mar 14, 2025 at 12:46 AM Thierry Volpiatto <thievol <at> posteo.net>
wrote:

> Ship Mints <shipmints <at> gmail.com> writes:
>
> >     Which bookmark handlers?
> >
> > At least these packages rely on bookmarks that store window state and
> > their handlers restore window state.  There are workarounds I have
> > considered but they are distasteful, to say the least.
> >
> > bufferlo
> > activities
> > burly
>
> After a quick look at source code of activities I see that it tries hard
> to fix the bug this change fixed, there is even comments, so I suggest
> these packages adapt their bookmark code to new bookmark--jump-via, IMO
> this will reduce the code and make it simpler.
>

Thanks for taking a look.  Those are just the ones that immediately came to
mind.  There may be many other users.  I stand by my suggestions that we
either or both:

- Add defcustoms and/or defvars to control this new behavior that breaks
bookmarks.  bookmarks are *not* documented to be merely for files, not
documented to avoid window and frame behavior (some of our bookmarks also
perform frame operations compounding the issue with this new behavior).

- Add bookmark-record properties/keys to control bookmark behavior so that
bookmark-record-make functions can decide for themselves which behavior to
trigger.

- Add a defustom to set the default display-func so that
pop-to-buffer-same-window can be overridden.  I consider that this is
missing as an oversight in the implementation that does not forbid common
use cases for which bookmarks have been adopted.

I'm happy to submit patches for this to make everyone's bookmark use more
general and less painful.  In the same vein, I've submitted patches to
correct an oversight in window-state-get that makes using bookmark-records
that contain window-state objects more reliable.

Don't you agree this is a better approach than a wholesale behavior change?
[Message part 2 (text/html, inline)]

This bug report was last modified 54 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.