unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 19841@debbugs.gnu.org
Subject: bug#19841: 25.0.50; Doc for `transient-mark-mode' should say that highlighting is with	face `region'
Date: Fri, 13 Feb 2015 06:52:56 -0800 (PST)	[thread overview]
Message-ID: <d79f6344-52f9-4f7e-a0fd-316970b7662f@default> (raw)
In-Reply-To: <<83oaoy41i4.fsf@gnu.org>>

> > Please update the doc string, the Emacs manual, node `Disabled Transient
> > Mark', and the Elisp manual, node `The Mark'.
> >
> > While you're at it, please so likewise for the Elisp manual node `The
> > Region'.  (At least Emacs manual node `Mark' mentions it.)
> >
> > It is not obvious to users which face they can customize to change the
> > `transient-mark-mode' highlighting.
> >
> > This thread is one indication of the need for this improvement:
> > http://lists.gnu.org/archive/html/help-gnu-emacs/2015-02/msg00236.html
> 
> I added references to the 'region' face to the 2 doc strings, but left
> the manuals alone.  We generally don't mention faces of features where
> those features are described; instead, there's a "Standard Faces" node
> in the Emacs manual that documents them all in one place.

OK, thank you.

Wouldn't it be good to xref node Standard Faces for this?
The doc string mention is sufficient, but someone reading
the manual might not think to also look at the doc string.

And in the case of `transient-mark-mode' and the `region'
face, the connection is not at all obvious.  For some other
modes and their faces the names are similar (e.g. if the
`region' were called `transient-mark' or something).  But
for these two, in particular, I think a cross-ref wouldn't
hurt.

Anyway, feel free to close the bug, and use your judgment
about the manuals.  I haven't looked at the new doc-string
text, but I'm sure it's a big improvement.





       reply	other threads:[~2015-02-13 14:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <<25162aaf-0573-4f31-a4bf-e8b64f8937ed@default>
     [not found] ` <<83oaoy41i4.fsf@gnu.org>
2015-02-13 14:52   ` Drew Adams [this message]
2015-02-13 15:10     ` bug#19841: 25.0.50; Doc for `transient-mark-mode' should say that highlighting is with face `region' Eli Zaretskii
2015-02-11 21:32 Drew Adams
2015-02-13  9:14 ` 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

  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=d79f6344-52f9-4f7e-a0fd-316970b7662f@default \
    --to=drew.adams@oracle.com \
    --cc=19841@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    /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).