all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#51876: 28.0.60; Confusing prompt for setting a bookmark
@ 2021-11-15 18:34 Eli Zaretskii
  2021-11-16  8:12 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 13+ messages in thread
From: Eli Zaretskii @ 2021-11-15 18:34 UTC (permalink / raw)
  To: 51876

To reproduce:

  emacs -Q
  C-x C-f some-file-name RET
  C-x r m

Emacs prompts thusly:

  Set bookmark unconditionally (default some-file-name):

This prompt is confusing: it doesn't make it clear that Emacs is
prompting for the _name_ of the bookmark.  The word "name" (or
anything to that effect) doesn't appear anywhere within the prompt,
and the suggested default, which is just the file name, makes the
intent even harder to guess, because it sounds like Emacs asks whether
to set a bookmark in this file.

Please make the prompt more user-friendly.

I also think the "unconditionally" part should be removed: it makes me
wonder why Emacs made a point of mentioning that at all.  It also
makes the prompt longer, thus harder to understand.

In GNU Emacs 28.0.60 (build 140, i686-pc-mingw32)
 of 2021-11-15 built on HOME-C4E4A596F7
Repository revision: b418aad85a3d62aa427e7af72c96ca1d644dbc02
Repository branch: emacs-28
Windowing system distributor 'Microsoft Corp.', version 5.1.2600
System Description: Microsoft Windows XP Service Pack 3 (v5.1.0.2600)

Configured using:
 'configure -C --prefix=/d/usr --with-wide-int --with-modules
 --enable-checking=yes,glyphs 'CFLAGS=-O0 -gdwarf-4 -g3''

Configured features:
ACL GIF GMP GNUTLS HARFBUZZ JPEG JSON LCMS2 LIBXML2 MODULES NOTIFY
W32NOTIFY PDUMPER PNG RSVG SOUND THREADS TIFF TOOLKIT_SCROLL_BARS XPM
ZLIB

Important settings:
  value of $LANG: ENU
  locale-coding-system: cp1255

Major mode: Lisp Interaction

Minor modes in effect:
  tooltip-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  show-paren-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  indent-tabs-mode: t
  transient-mark-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message rmc puny dired dired-loaddefs
rfc822 mml mml-sec epa derived epg rfc6068 epg-config gnus-util rmail
rmail-loaddefs auth-source cl-seq eieio eieio-core cl-macs
eieio-loaddefs password-cache json map text-property-search time-date
subr-x seq byte-opt gv bytecomp byte-compile cconv mm-decode mm-bodies
mm-encode mail-parse rfc2231 mailabbrev gmm-utils mailheader cl-loaddefs
cl-lib sendmail rfc2047 rfc2045 ietf-drums mm-util mail-prsvr mail-utils
iso-transl tooltip eldoc paren electric uniquify ediff-hook vc-hooks
lisp-float-type elisp-mode mwheel dos-w32 ls-lisp disp-table
term/w32-win w32-win w32-vars term/common-win tool-bar dnd fontset image
regexp-opt fringe tabulated-list replace newcomment text-mode lisp-mode
prog-mode register page tab-bar menu-bar rfn-eshadow isearch easymenu
timer select scroll-bar mouse jit-lock font-lock syntax font-core
term/tty-colors frame minibuffer cl-generic cham georgian utf-8-lang
misc-lang vietnamese tibetan thai tai-viet lao korean japanese eucjp-ms
cp51932 hebrew greek romanian slovak czech european ethiopic indian
cyrillic chinese composite emoji-zwj charscript charprop case-table
epa-hook jka-cmpr-hook help simple abbrev obarray cl-preloaded nadvice
button loaddefs faces cus-face macroexp files window text-properties
overlay sha1 md5 base64 format env code-pages mule custom widget
hashtable-print-readable backquote threads w32notify w32 lcms2 multi-tty
make-network-process emacs)

Memory information:
((conses 16 56698 6812)
 (symbols 48 7801 1)
 (strings 16 21653 2791)
 (string-bytes 1 632658)
 (vectors 16 13628)
 (vector-slots 8 180022 11936)
 (floats 8 23 93)
 (intervals 40 266 116)
 (buffers 888 10))






Set bookmark unconditionally (default bookmark-test.txt):





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-15 18:34 bug#51876: 28.0.60; Confusing prompt for setting a bookmark Eli Zaretskii
@ 2021-11-16  8:12 ` Lars Ingebrigtsen
  2021-11-16 13:12   ` Eli Zaretskii
  0 siblings, 1 reply; 13+ messages in thread
From: Lars Ingebrigtsen @ 2021-11-16  8:12 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 51876

Eli Zaretskii <eliz@gnu.org> writes:

> This prompt is confusing: it doesn't make it clear that Emacs is
> prompting for the _name_ of the bookmark.  The word "name" (or
> anything to that effect) doesn't appear anywhere within the prompt,
> and the suggested default, which is just the file name, makes the
> intent even harder to guess, because it sounds like Emacs asks whether
> to set a bookmark in this file.

Yes.

> I also think the "unconditionally" part should be removed: it makes me
> wonder why Emacs made a point of mentioning that at all.  It also
> makes the prompt longer, thus harder to understand.

It's because of this:

---
With a prefix arg (non-nil NO-OVERWRITE), do not overwrite any
existing bookmark that has the same name as NAME, but instead push the
new bookmark onto the bookmark alist.
---

Perhaps 

         (if no-overwrite "Set bookmark" "Set bookmark unconditionally")))

should be

         (if no-overwrite "Append bookmark" "Set bookmark")))


-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-16  8:12 ` Lars Ingebrigtsen
@ 2021-11-16 13:12   ` Eli Zaretskii
  2021-11-17  6:22     ` Stefan Kangas
  0 siblings, 1 reply; 13+ messages in thread
From: Eli Zaretskii @ 2021-11-16 13:12 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 51876

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Cc: 51876@debbugs.gnu.org
> Date: Tue, 16 Nov 2021 09:12:23 +0100
> 
> Perhaps 
> 
>          (if no-overwrite "Set bookmark" "Set bookmark unconditionally")))
> 
> should be
> 
>          (if no-overwrite "Append bookmark" "Set bookmark")))

I would suggest

  (if no-overwrite "Add bookmark named" "Set bookmark named")





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-16 13:12   ` Eli Zaretskii
@ 2021-11-17  6:22     ` Stefan Kangas
  2021-11-17  7:17       ` Lars Ingebrigtsen
                         ` (2 more replies)
  0 siblings, 3 replies; 13+ messages in thread
From: Stefan Kangas @ 2021-11-17  6:22 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Lars Ingebrigtsen, 51876

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Lars Ingebrigtsen <larsi@gnus.org>
>> Cc: 51876@debbugs.gnu.org
>> Date: Tue, 16 Nov 2021 09:12:23 +0100
>>
>> Perhaps
>>
>>          (if no-overwrite "Set bookmark" "Set bookmark unconditionally")))
>>
>> should be
>>
>>          (if no-overwrite "Append bookmark" "Set bookmark")))
>
> I would suggest
>
>   (if no-overwrite "Add bookmark named" "Set bookmark named")

How about

    (if no-overwrite "Append bookmark named" "Set bookmark named")

?





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-17  6:22     ` Stefan Kangas
@ 2021-11-17  7:17       ` Lars Ingebrigtsen
  2021-11-17 16:43         ` bug#51876: [External] : " Drew Adams
  2021-11-17 13:50       ` Eli Zaretskii
  2021-11-17 16:42       ` Drew Adams
  2 siblings, 1 reply; 13+ messages in thread
From: Lars Ingebrigtsen @ 2021-11-17  7:17 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 51876

Stefan Kangas <stefan@marxist.se> writes:

> How about
>
>     (if no-overwrite "Append bookmark named" "Set bookmark named")

And we have a winner.  Pushed to Emacs 29 now.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-17  6:22     ` Stefan Kangas
  2021-11-17  7:17       ` Lars Ingebrigtsen
@ 2021-11-17 13:50       ` Eli Zaretskii
  2021-11-17 16:49         ` bug#51876: [External] : " Drew Adams
  2021-11-18  9:21         ` Lars Ingebrigtsen
  2021-11-17 16:42       ` Drew Adams
  2 siblings, 2 replies; 13+ messages in thread
From: Eli Zaretskii @ 2021-11-17 13:50 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: larsi, 51876

> From: Stefan Kangas <stefan@marxist.se>
> Date: Tue, 16 Nov 2021 22:22:59 -0800
> Cc: Lars Ingebrigtsen <larsi@gnus.org>, 51876@debbugs.gnu.org
> 
> > I would suggest
> >
> >   (if no-overwrite "Add bookmark named" "Set bookmark named")
> 
> How about
> 
>     (if no-overwrite "Append bookmark named" "Set bookmark named")
> 
> ?

Not that it matters now, but I'm curious: why "append" and not "add"?





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: [External] : bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-17  6:22     ` Stefan Kangas
  2021-11-17  7:17       ` Lars Ingebrigtsen
  2021-11-17 13:50       ` Eli Zaretskii
@ 2021-11-17 16:42       ` Drew Adams
  2021-11-17 17:08         ` Eli Zaretskii
  2 siblings, 1 reply; 13+ messages in thread
From: Drew Adams @ 2021-11-17 16:42 UTC (permalink / raw)
  To: Stefan Kangas, Eli Zaretskii; +Cc: Lars Ingebrigtsen, 51876@debbugs.gnu.org

> > I would suggest
> > (if no-overwrite "Add bookmark named" "Set bookmark named")
> 
> How about
> (if no-overwrite "Append bookmark named" "Set bookmark named")

This would be quite wrong.  The new bookmark is
prepended to the bookmark alist, not appended.

The point is that you can have multiple bookmarks
with the same name.  And as is usual with alists,
a bookmark closer to the front of the alist
shadows those farther from the front that have
the same name.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: [External] : bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-17  7:17       ` Lars Ingebrigtsen
@ 2021-11-17 16:43         ` Drew Adams
  0 siblings, 0 replies; 13+ messages in thread
From: Drew Adams @ 2021-11-17 16:43 UTC (permalink / raw)
  To: Lars Ingebrigtsen, Stefan Kangas; +Cc: 51876@debbugs.gnu.org

> > How about
> > (if no-overwrite "Append bookmark named" "Set bookmark named")
> 
> And we have a winner.  Pushed to Emacs 29 now.

And we have a loser: Emacs 29.





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: [External] : bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-17 13:50       ` Eli Zaretskii
@ 2021-11-17 16:49         ` Drew Adams
  2021-11-18  9:21         ` Lars Ingebrigtsen
  1 sibling, 0 replies; 13+ messages in thread
From: Drew Adams @ 2021-11-17 16:49 UTC (permalink / raw)
  To: Eli Zaretskii, Stefan Kangas; +Cc: larsi@gnus.org, 51876@debbugs.gnu.org

> > > I would suggest
> > > (if no-overwrite "Add bookmark named" "Set bookmark named")
> > How about
> > (if no-overwrite "Append bookmark named" "Set bookmark named")
> 
> Not that it matters now, but I'm curious:
> why "append" and not "add"?

It does matter, because "append" is just wrong.

But perhaps, by "now", you meant that it's too
late to fix this?  I hope not.





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: [External] : bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-17 16:42       ` Drew Adams
@ 2021-11-17 17:08         ` Eli Zaretskii
  2021-11-17 17:46           ` Drew Adams
  0 siblings, 1 reply; 13+ messages in thread
From: Eli Zaretskii @ 2021-11-17 17:08 UTC (permalink / raw)
  To: Drew Adams; +Cc: larsi, stefan, 51876

> From: Drew Adams <drew.adams@oracle.com>
> CC: Lars Ingebrigtsen <larsi@gnus.org>,
>         "51876@debbugs.gnu.org"
> 	<51876@debbugs.gnu.org>
> Date: Wed, 17 Nov 2021 16:42:38 +0000
> 
> The point is that you can have multiple bookmarks
> with the same name.

Yes, but can you say that in few enough characters to fit a prompt, so
that it is self-explanatory and yet short?  That's the challenge;
everything else is a tangent, since it isn't like people here don't
understand what that does.





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: [External] : bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-17 17:08         ` Eli Zaretskii
@ 2021-11-17 17:46           ` Drew Adams
  0 siblings, 0 replies; 13+ messages in thread
From: Drew Adams @ 2021-11-17 17:46 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: larsi@gnus.org, stefan@marxist.se, 51876@debbugs.gnu.org

> > The point is that you can have multiple bookmarks
> > with the same name.
> 
> Yes, but can you say that in few enough characters to fit a prompt, so
> that it is self-explanatory and yet short?  That's the challenge;
> everything else is a tangent, since it isn't like people here don't
> understand what that does.

I think you already said it in few chars: "Add",
not "Append".  "Add" is fine; "Append" is wrong.

Someone who suggests putting "append" in the
prompt seems to me not to understand well what
the command does.

Or else I just misunderstand their understanding.
In any case, to my mind "append" is wrong.  It
will only mislead or confuse users.

For vanilla Emacs, only one bookmark with a given
name can be used at any time.  Because of that, I
think "set" is a fine term to use.

The suggestion of "add" instead is OK too, but it
can wrongly suggest that other bookmarks with the
same name can be used at the same time.

And you'll note that "add" agrees with what's said
in the doc string of `bookmark-set-no-overwrite',
except there the word used is "push".

"Push" is thus another reasonable alternative
(like "add") to the incorrect "append".  Even
"prepend" more or less correct, but it's not as
good as "push", "add" (better than "push", or
"set" (best of all, IMO).
___

And speaking of (real) tangents -

This bug thread perhaps shouldn't have gone into
the weeds to deal with something different from
what you reported, which was, I think, a concern
that it isn't clear enough that what's prompted
for is a _bookmark_ name (not, e.g., a file name).
___

FWIW -

1. Bookmark+ uses the name
`bmkp-bookmark-set-confirm-overwrite' for a command
that requires confirmation before overwriting an
existing bookmark (unless you provide a prefix arg).

I prefer that name and behavior to what vanilla
Emacs added as command `bookmark-set-no-overwrite'.

2. With Bookmark+, it _is_ possible to use multiple
bookmarks that have the same name at the same time.
It's possible because the full bookmark is kept on
the bookmark name as a property.

This optional behavior is useful, for example to
(automatically) get bookmark names that are non-dir
file names, to files with the same name in different
directories.





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-17 13:50       ` Eli Zaretskii
  2021-11-17 16:49         ` bug#51876: [External] : " Drew Adams
@ 2021-11-18  9:21         ` Lars Ingebrigtsen
  2021-11-18 17:08           ` bug#51876: [External] : " Drew Adams
  1 sibling, 1 reply; 13+ messages in thread
From: Lars Ingebrigtsen @ 2021-11-18  9:21 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 51876, Stefan Kangas

Eli Zaretskii <eliz@gnu.org> writes:

> Not that it matters now, but I'm curious: why "append" and not "add"?

With "append" it's clear that we're talking about a list of names, while
"add" is ambiguous. 

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 13+ messages in thread

* bug#51876: [External] : bug#51876: 28.0.60; Confusing prompt for setting a bookmark
  2021-11-18  9:21         ` Lars Ingebrigtsen
@ 2021-11-18 17:08           ` Drew Adams
  0 siblings, 0 replies; 13+ messages in thread
From: Drew Adams @ 2021-11-18 17:08 UTC (permalink / raw)
  To: Lars Ingebrigtsen, Eli Zaretskii; +Cc: 51876@debbugs.gnu.org, Stefan Kangas

> > Not that it matters now, but I'm curious: why "append" and not "add"?
> 
> With "append" it's clear that we're talking about a list of names, while
> "add" is ambiguous.

We're _not_ appending.  Not at all.

We're actually _setting_ the bookmark with
that name to the specified (typically new)
location.  "Set" is the right word.  It's
precisely the word for what happens.

It's true that wrt the `bookmark-alist'
we're "adding" the bookmark to the front
of the alist ("prepending", if you like).

But we're not adding a bookmark to those
that are in effect at the current time.
Once the new bookmark is "added" this way,
any previously existing bookmarks with the
same name are _no longer usable_ (till the
new one is removed).  They _exist_ on the
alist (users can manipulate them there,
e.g., using Lisp), but they're not usable
as such (you can't jump to them, etc.).

So far, you're just making things worse,
I'm afraid.

`bookmark-set' sets a bookmark.  "Set"
means create or update.

That there are 2 ways to deal with
previously created bookmarks with the same
name is a different question, and is best
dealt with by having two commands, one of
which prompts for confirmation to overwrite
(see my previous msg).





^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2021-11-18 17:08 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-11-15 18:34 bug#51876: 28.0.60; Confusing prompt for setting a bookmark Eli Zaretskii
2021-11-16  8:12 ` Lars Ingebrigtsen
2021-11-16 13:12   ` Eli Zaretskii
2021-11-17  6:22     ` Stefan Kangas
2021-11-17  7:17       ` Lars Ingebrigtsen
2021-11-17 16:43         ` bug#51876: [External] : " Drew Adams
2021-11-17 13:50       ` Eli Zaretskii
2021-11-17 16:49         ` bug#51876: [External] : " Drew Adams
2021-11-18  9:21         ` Lars Ingebrigtsen
2021-11-18 17:08           ` bug#51876: [External] : " Drew Adams
2021-11-17 16:42       ` Drew Adams
2021-11-17 17:08         ` Eli Zaretskii
2021-11-17 17:46           ` Drew Adams

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.