unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: "Göktuğ Kayaalp" <self@gkayaalp.com>
Cc: eggert@cs.ucla.edu, emacs-devel@gnu.org
Subject: Re: [PATCH] Enable customisation for electric-quote-mode chars
Date: Mon, 29 Aug 2016 18:08:25 +0300	[thread overview]
Message-ID: <83fupnbok6.fsf@gnu.org> (raw)
In-Reply-To: <87d1ksdtzg.fsf@xi.bootis> (message from Göktuğ Kayaalp on Mon, 29 Aug 2016 08:28:19 +0300)

> From: Göktuğ Kayaalp <self@gkayaalp.com>
> Cc: eliz@gnu.org, emacs-devel@gnu.org
> Date: Mon, 29 Aug 2016 08:28:19 +0300
> 
> +@vindex electric-quote-chars
>    Electric Quote mode makes it easier to type curved quotes.  As you
>  type characters it optionally converts @t{`} to @t{‘}, @t{'} to @t{’},
>  @t{``} to @t{“}, and @t{''} to @t{”}.  These conversions are
>  suppressed in buffers whose coding systems cannot represent curved
> -quote characters.
> +quote characters.  It's possible to change the default quotes listed
> +above, by customizing the variable @code{electric-quote-chars}, a list
> +of four characters, where the items correspond to the left single
> +quote, the right single quote, the left double quote and the right
> +double quote, respectively, whose default value is
> +@code{'(?‘ ?’ ?“ ?”)}.

There's no need to go to such detail when describing a customizable
option.  The Customize interface should be enough for the user to
figure out what to do.

Which brings me to this:

> +(defcustom electric-quote-chars '(?‘ ?’ ?“ ?”)
> +  "Curved quote characters for `electric-quote-mode'.
> +The items correspond to the left single quote, the right single
> +quote, the left double quote, and the right double quote, respectively."
> +  :version "25.1"
> +  :type 'list :safe 'listp :group 'electricity)
> +

Making it a list with minimal explanations sounds like not very
user-friendly.  How about offering several ready out-of-the-box sets,
each one for some culture we know about?  We could also allow a raw
list, for the power users, but I think less advanced Emacs users will
appreciate something like

  Default (‘..’ “..”)
  Arabic (".." ”..“)
  German  (‚..‛ „..“)
  French (‹..› «..»)
  Cyrillic (‚..‘ «..»)

etc.  WDYT?  (See https://en.wikipedia.org/wiki/Quotation_mark for
some more.)

Thanks.



  parent reply	other threads:[~2016-08-29 15:08 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-27 14:22 [PATCH] Enable customisation for electric-quote-mode chars Göktuğ Kayaalp
2016-08-27 14:38 ` Eli Zaretskii
2016-08-27 15:23   ` Göktuğ Kayaalp
2016-08-27 19:16     ` Paul Eggert
2016-08-28  1:00       ` Göktuğ Kayaalp
2016-08-29  1:55         ` Paul Eggert
2016-08-29  5:28           ` Göktuğ Kayaalp
2016-08-29  6:14             ` Paul Eggert
2016-10-05 18:53               ` Göktuğ Kayaalp
2016-10-05 19:06                 ` Paul Eggert
2016-10-06  6:40                   ` Eli Zaretskii
2016-10-06 21:31                     ` Paul Eggert
2016-10-07 18:33                       ` Göktuğ Kayaalp
2016-10-10  3:57                         ` Paul Eggert
2016-10-13 18:28                           ` Göktuğ Kayaalp
2016-10-13 18:35                             ` Paul Eggert
2016-10-22 14:00                               ` Göktuğ Kayaalp
2016-10-23 10:25                                 ` Paul Eggert
2016-10-23 15:09                                   ` Göktuğ Kayaalp
2016-10-27 15:12                                     ` Paul Eggert
2016-10-27 17:21                                       ` Göktuğ Kayaalp
2016-08-29 15:08             ` Eli Zaretskii [this message]
2016-08-29 15:54               ` Yuri Khan
2016-08-29 16:23                 ` Eli Zaretskii
2016-08-29 16:27               ` Göktuğ Kayaalp
2016-08-29 16:39                 ` Eli Zaretskii
2016-08-29 17:19                   ` Göktuğ Kayaalp
2016-08-29 17:30                 ` Paul Eggert
2016-08-29 17:44                   ` Eli Zaretskii
2016-08-29 18:43                     ` Paul Eggert
2016-08-29 19:30                       ` Eli Zaretskii
2016-08-30 17:38                         ` Paul Eggert
2016-08-30 17:49                           ` Eli Zaretskii
2016-08-31 11:08                             ` Richard Stallman
2016-09-01 18:56                         ` Göktuğ Kayaalp
2016-09-01 19:15                           ` Paul Eggert
2016-09-01 21:13                             ` Göktuğ Kayaalp
2016-09-01 21:30                               ` Paul Eggert
2016-09-02  5:06                                 ` Yuri Khan
2016-09-02  7:30                                   ` Eli Zaretskii
2016-09-02 10:37                                     ` Yuri Khan
2016-09-02 13:24                                       ` Göktuğ Kayaalp
2016-08-29 16:15           ` tarball builds (was: [PATCH] Enable customisation for electric-quote-mode chars) Stefan Monnier
2016-08-30 15:14             ` Eli Zaretskii
2016-08-30 15:53               ` tarball builds Stefan Monnier
2016-08-30 15:59             ` Paul Eggert
2016-08-30 17:00               ` Stefan Monnier
2016-08-30 17:58                 ` Paul Eggert
2016-08-29  2:33         ` [PATCH] Enable customisation for electric-quote-mode chars 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=83fupnbok6.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=eggert@cs.ucla.edu \
    --cc=emacs-devel@gnu.org \
    --cc=self@gkayaalp.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 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).