all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: Juri Linkov <juri@linkov.net>, Tino Calancha <tino.calancha@gmail.com>
Cc: 30285@debbugs.gnu.org, jidanni@jidanni.org
Subject: bug#30285: dired-do-chmod vs. top line of dired
Date: Thu, 1 Feb 2018 12:50:16 -0800 (PST)	[thread overview]
Message-ID: <5620cf82-9402-4895-bdd1-fdf5401c8172@default> (raw)
In-Reply-To: <87k1vwcwy1.fsf@mail.linkov.net>

> Since `dired-mark' from the top line followed by the command in question
> is not obvious for users, we could provide a hint in the error message,
> i.e. mention the availability of ‘m’ on the top line with such message:
> “You can type `m' here to mark all files for this operation”.

Not sure how helpful or necessary that is.  It's liable to
not be helpful (that use case needs no special advertising).
And it might even confuse things.  I think it just gets in
the way of the message, which is, "You are not on a file line."

I think we should show the same error message from any non-file
line - not just the top line or top-two lines.  And I think it
should just say that point is not on a file line.

Users who are interested or who wonder about the error message
can consult the doc string of the command.

That's the place, if any is needed, where such info as you
mention should be conveyed.  Such info is not part of the
message we're trying to get across here, which is just that
this command cannot be used if point is not on a file line.

Dired lets you do all kinds of things.  There are lots of ways
top mark files and lots of ways to act on marked files.  This
command should not be advertising any such ways in its error
message.

> I agree with Drew that better to use existing functions, and not to
> duplicate them.  Non sunt multiplicanda entia sine necessitate.

;-)

> Moreover, we should not change the old semantic of Dired commands:
> if users have a habit of operating on the first files by going to the
> top
> line and typing e.g. ‘M-< C-5 M’ to change modes of the first 5 files,
> this is just fine, we should not prohobit this behaviour now.
> 
> So what we need to do is just check if the list of files returned from
> ‘dired-get-marked-files’ is nil, and show some message in this case in
> all places that you found where the prompt with [0 files] makes no
> sense.

Yes.  Just a simple fix for this minor usability bug, please.





  reply	other threads:[~2018-02-01 20:50 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-29 12:32 bug#30285: dired-do-chmod vs. top line of dired 積丹尼 Dan Jacobson
2018-01-29 15:14 ` Tino Calancha
2018-01-29 16:05   ` Eli Zaretskii
2018-01-29 23:21     ` Tino Calancha
2018-01-29 23:42       ` Drew Adams
2018-01-30  3:53         ` Tino Calancha
2018-01-30  4:43           ` Drew Adams
2018-01-30 15:15             ` Drew Adams
2018-01-31  9:49               ` Tino Calancha
2018-01-31 19:04                 ` Drew Adams
2018-01-31 21:35         ` Juri Linkov
2018-01-31 23:20           ` Drew Adams
2018-02-01  8:16           ` Tino Calancha
2018-02-01  9:17             ` Tino Calancha
2018-02-01 16:10             ` Drew Adams
2018-02-04 23:12               ` Tino Calancha
2018-02-05 16:45                 ` Drew Adams
2018-02-01 20:07             ` Juri Linkov
2018-02-01 20:50               ` Drew Adams [this message]
2018-02-01 21:35                 ` Juri Linkov
2018-02-01 22:23                   ` Drew Adams
2018-02-03 22:23                     ` Juri Linkov
2018-02-04 10:02                       ` martin rudalics
2018-02-04 21:44                         ` Juri Linkov
2018-02-06 21:32                         ` Juri Linkov
2018-02-04 23:08                   ` Tino Calancha
2018-02-05 21:01                     ` Juri Linkov
2018-02-05 21:52                       ` Drew Adams
2018-01-29 15:24 ` 積丹尼 Dan Jacobson
2018-01-29 23:14   ` Tino Calancha

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=5620cf82-9402-4895-bdd1-fdf5401c8172@default \
    --to=drew.adams@oracle.com \
    --cc=30285@debbugs.gnu.org \
    --cc=jidanni@jidanni.org \
    --cc=juri@linkov.net \
    --cc=tino.calancha@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 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.