all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Drew Adams <drew.adams@oracle.com>
Cc: 55050-done@debbugs.gnu.org
Subject: bug#55050: 27.2; Doc strings for commands that do query-replacements
Date: Thu, 21 Apr 2022 15:03:24 +0300	[thread overview]
Message-ID: <83v8v2r6cz.fsf@gnu.org> (raw)
In-Reply-To: <SJ0PR10MB5488E770D0948DFA9421F699F3F49@SJ0PR10MB5488.namprd10.prod.outlook.com> (message from Drew Adams on Thu, 21 Apr 2022 02:46:16 +0000)

> From: Drew Adams <drew.adams@oracle.com>
> Date: Thu, 21 Apr 2022 02:46:16 +0000
> 
> This is about the doc of commands, such as
> `dired-do-find-regexp-and-replace', that use a query-replace dialog.
> There are a bunch of such commands now.  For the most part, they don't
> tell you anything about what to expect, in terms of a dialog.  In
> particular, they don't reference the doc of `query-replace`, which is
> (at least) where the dialog should be described.
> 
> The only saving grace is that there's a prompt that suggests that if you
> hit `?' you might get some information about what you can do.  Assuming
> you notice that prompt, of course.

If you don't assume they pay attention to the prompt, why assume they
will read the doc string, which is much longer and whose instructions
it is therefore much easier to miss?

> The doc of all such commands should, directly or indirectly, tell you
> about the keys you can use and what they do - what you learn by hitting
> `?'.

It is not reasonable to show all the commands available for dealing
with matches: the list is very long, and will make the doc strings
unbearably long, especially for people who are interested in other
aspects of these commands than how to deal with matches.

So I added the minimal instructions to all the commands I found which
perform query-replace of some sort: how to accept a replacement and
how to skip it -- with a reference to 'h' that will show the full
instructions.  This, of course, is _in_addition_ to the already
existing link to 'query-replace'/'query-replace-regexp', which in
itself is a reference to that stuff.

> See, for example, this user's question:
> 
> https://emacs.stackexchange.com/q/71451/105

FTR, that question has nothing to do with the subject of this bug
report, it is about the user's confusion with the fact that Emacs
doesn't let users modify write-protected files, and what to do when
the user wants to modify such files even though they are read-only.





  parent reply	other threads:[~2022-04-21 12:03 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-21  2:46 bug#55050: 27.2; Doc strings for commands that do query-replacements Drew Adams
2022-04-21 11:50 ` Lars Ingebrigtsen
2022-04-21 12:03 ` Eli Zaretskii [this message]
2022-04-21 16:04   ` Juri Linkov
2022-04-21 18:57     ` Eli Zaretskii

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83v8v2r6cz.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=55050-done@debbugs.gnu.org \
    --cc=drew.adams@oracle.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 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.