unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#16822: 24.3.50; show-paren-mode adds confusion to active region
@ 2014-02-20  7:40 yynyygy
  2014-02-20 17:44 ` Drew Adams
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: yynyygy @ 2014-02-20  7:40 UTC (permalink / raw)
  To: 16822


When show-paren-mode is on, it adds confusion to the active
region. Suppose I have some text which is (hello),

1. I set the region to (hello and place the cursor on the left paren.
2. I set the region to (hello) and place the cursor on the left paren.

Note that in Case 1, the right paren is not part of the region while in
Case 2 it is. In the two cases above, I get exactly the same color on
the screen, then how can I distinguish between these two different
cases?

Compared to the more rational behavior of Emacs 24.3, I think the paren
which is part of the region should have the same color with the region
while the paren which is not part of the region can has matching paren
color. In this way, the user can tell which part is selected and which
part is not.



In GNU Emacs 24.3.50.1 (i686-pc-linux-gnu, GTK+ Version 3.8.7)
 of 2014-02-19 on gentoo
Windowing system distributor `The X.Org Foundation', version 11.0.11403000
Configured using:
 `configure --prefix=/usr/local/emacs/'

Important settings:
  value of $LC_COLLATE: C
  value of $LC_CTYPE: zh_CN.UTF-8
  value of $LANG: en_US.UTF-8
  value of $XMODIFIERS: @im=ibus
  locale-coding-system: utf-8-unix

Major mode: Messages

Minor modes in effect:
  tooltip-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
  buffer-read-only: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
M-x r e p o <tab> r <tab> <return> M <backspace> s 
h o w - p a r e n - m o d e SPC a d d SPC c o n f <backspace> 
<backspace> <backspace> <backspace> <backspace> s SPC 
c o n f u s i o C-g C-h f t r a n <tab> s <backspace> 
i <tab> <return> C-x o C-n C-n C-n C-n C-n C-n C-n 
C-l C-n C-n C-n C-n C-n C-n C-l C-n C-n C-n C-n C-l 
C-n C-n C-n C-l C-n C-n C-n C-l q M-x r e C-g C-x b 
<tab> <return> M-x r e p o <tab> r <tab> <return>

Recent messages:
For information about GNU Emacs and the GNU system, type C-h C-a.
Making completion list...
Quit
Type C-x 1 to delete the help window, C-M-v to scroll help.
Quit
Making completion list... [2 times]

Load-path shadows:
None found.

Features:
(shadow sort gnus-util mail-extr emacsbug message format-spec rfc822 mml
mml-sec mm-decode mm-bodies mm-encode mail-parse rfc2231 mailabbrev
gmm-utils mailheader sendmail rfc2047 rfc2045 ietf-drums mm-util
help-fns mail-prsvr mail-utils help-mode easymenu time-date china-util
tooltip electric uniquify ediff-hook vc-hooks lisp-float-type mwheel
x-win x-dnd tool-bar dnd fontset image regexp-opt fringe tabulated-list
newcomment lisp-mode prog-mode register page menu-bar rfn-eshadow timer
select scroll-bar mouse jit-lock font-lock syntax facemenu font-core
frame cham georgian utf-8-lang misc-lang vietnamese tibetan thai
tai-viet lao korean japanese hebrew greek romanian slovak czech european
ethiopic indian cyrillic chinese case-table epa-hook jka-cmpr-hook help
simple abbrev minibuffer nadvice loaddefs button faces cus-face macroexp
files text-properties overlay sha1 md5 base64 format env code-pages mule
custom widget hashtable-print-readable backquote make-network-process
dbusbind gfilenotify dynamic-setting system-font-setting
font-render-setting move-toolbar gtk x-toolkit x multi-tty emacs)





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

* bug#16822: 24.3.50; show-paren-mode adds confusion to active region
  2014-02-20  7:40 bug#16822: 24.3.50; show-paren-mode adds confusion to active region yynyygy
@ 2014-02-20 17:44 ` Drew Adams
  2014-02-20 21:43   ` Stefan Monnier
  2021-07-15  4:54 ` Lars Ingebrigtsen
  2021-07-15  4:59 ` Lars Ingebrigtsen
  2 siblings, 1 reply; 7+ messages in thread
From: Drew Adams @ 2014-02-20 17:44 UTC (permalink / raw)
  To: yynyygy, 16822

> When show-paren-mode is on, it adds confusion to the active
> region. Suppose I have some text which is (hello),
> 
> 1. I set the region to (hello and place the cursor on the left paren.
> 2. I set the region to (hello) and place the cursor on the left paren.
> 
> Note that in Case 1, the right paren is not part of the region while in
> Case 2 it is. In the two cases above, I get exactly the same color on
> the screen, then how can I distinguish between these two different
> cases?
> 
> Compared to the more rational behavior of Emacs 24.3, I think the paren
> which is part of the region should have the same color with the region
> while the paren which is not part of the region can has matching paren
> color. In this way, the user can tell which part is selected and which
> part is not.

Well put.

Yes, it used to be the case that the region highlighting was used
to show clearly what text is in the region (each char), as specified
by the doc:

http://www.gnu.org/software/emacs/manual/html_node/elisp/Displaying-Faces.html

That policy was changed recently, introducing the behavior change
that you report.  Now, other highlighting can take precedence
visually over region highlighting for various chars of the region.

Some people like the change.  Some don't.  Regression or improvement -
take your pick.  (No, you cannot take your pick in practice, e.g.,
via customization.  But you can at least make up your own mind.)

If you are interested, see bugs #15899, #16192, #15618, and this very
long emacs-devel thread:
http://lists.gnu.org/archive/html/emacs-devel/2014-01/msg01272.html

15899: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=15899
16192: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=16192
15618: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=15618





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

* bug#16822: 24.3.50; show-paren-mode adds confusion to active region
  2014-02-20 17:44 ` Drew Adams
@ 2014-02-20 21:43   ` Stefan Monnier
  0 siblings, 0 replies; 7+ messages in thread
From: Stefan Monnier @ 2014-02-20 21:43 UTC (permalink / raw)
  To: Drew Adams; +Cc: yynyygy, 16822

> take your pick.  (No, you cannot take your pick in practice, e.g.,
> via customization.  But you can at least make up your own mind.)

Maybe not via M-x customize, indeed, but since it's now written in
Elisp, you definitely can change it in your ~/.emacs.


        Stefan





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

* bug#16822: 24.3.50; show-paren-mode adds confusion to active region
  2014-02-20  7:40 bug#16822: 24.3.50; show-paren-mode adds confusion to active region yynyygy
  2014-02-20 17:44 ` Drew Adams
@ 2021-07-15  4:54 ` Lars Ingebrigtsen
  2021-07-15  4:59 ` Lars Ingebrigtsen
  2 siblings, 0 replies; 7+ messages in thread
From: Lars Ingebrigtsen @ 2021-07-15  4:54 UTC (permalink / raw)
  To: yynyygy; +Cc: 16822

yynyygy@gmail.com writes:

> When show-paren-mode is on, it adds confusion to the active
> region. Suppose I have some text which is (hello),
>
> 1. I set the region to (hello and place the cursor on the left paren.
> 2. I set the region to (hello) and place the cursor on the left paren.
>
> Note that in Case 1, the right paren is not part of the region while in
> Case 2 it is. In the two cases above, I get exactly the same color on
> the screen, then how can I distinguish between these two different
> cases?



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





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

* bug#16822: 24.3.50; show-paren-mode adds confusion to active region
  2014-02-20  7:40 bug#16822: 24.3.50; show-paren-mode adds confusion to active region yynyygy
  2014-02-20 17:44 ` Drew Adams
  2021-07-15  4:54 ` Lars Ingebrigtsen
@ 2021-07-15  4:59 ` Lars Ingebrigtsen
  2021-07-15  6:53   ` Eli Zaretskii
  2 siblings, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2021-07-15  4:59 UTC (permalink / raw)
  To: yynyygy; +Cc: 16822

[-- Attachment #1: Type: text/plain, Size: 684 bytes --]

yynyygy@gmail.com writes:

> When show-paren-mode is on, it adds confusion to the active
> region. Suppose I have some text which is (hello),
>
> 1. I set the region to (hello and place the cursor on the left paren.
> 2. I set the region to (hello) and place the cursor on the left paren.
>
> Note that in Case 1, the right paren is not part of the region while in
> Case 2 it is. In the two cases above, I get exactly the same color on
> the screen, then how can I distinguish between these two different
> cases?

This is still the case in Emacs 28.  Whether I put the mark after the
"o" or after the ")", and then go to the start of the line (in
*scratch*), I get this displayed:


[-- Attachment #2: Type: image/png, Size: 1601 bytes --]

[-- Attachment #3: Type: text/plain, Size: 218 bytes --]


Perhaps the solution here is just to introduce a new face for parens
that are part of the region?  Any opinions?

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

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

* bug#16822: 24.3.50; show-paren-mode adds confusion to active region
  2021-07-15  4:59 ` Lars Ingebrigtsen
@ 2021-07-15  6:53   ` Eli Zaretskii
  2021-07-15  8:45     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 7+ messages in thread
From: Eli Zaretskii @ 2021-07-15  6:53 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: yynyygy, 16822

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Date: Thu, 15 Jul 2021 06:59:21 +0200
> Cc: 16822@debbugs.gnu.org
> 
> yynyygy@gmail.com writes:
> 
> > When show-paren-mode is on, it adds confusion to the active
> > region. Suppose I have some text which is (hello),
> >
> > 1. I set the region to (hello and place the cursor on the left paren.
> > 2. I set the region to (hello) and place the cursor on the left paren.
> >
> > Note that in Case 1, the right paren is not part of the region while in
> > Case 2 it is. In the two cases above, I get exactly the same color on
> > the screen, then how can I distinguish between these two different
> > cases?
> 
> This is still the case in Emacs 28.  Whether I put the mark after the
> "o" or after the ")", and then go to the start of the line (in
> *scratch*), I get this displayed:
> [...]
> Perhaps the solution here is just to introduce a new face for parens
> that are part of the region?  Any opinions?

I admit that I don't understand the problem.  Why should the display
be different in these two cases?  It sounds like the (unspelled-out)
assumption is that showing the region is more important than showing
the (mis)matched parens?  But if this is the assumption, then I don't
think I agree: show-paren-mode is for _temporary_ display of the
parens, so if it temporarily obscures the region, it's perfectly okay.

Adding a new face would bump into the problem of making sure the
colors of this new face are always visible and distinguishable from
the other two colors.  I don't see how that could work reliably.





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

* bug#16822: 24.3.50; show-paren-mode adds confusion to active region
  2021-07-15  6:53   ` Eli Zaretskii
@ 2021-07-15  8:45     ` Lars Ingebrigtsen
  0 siblings, 0 replies; 7+ messages in thread
From: Lars Ingebrigtsen @ 2021-07-15  8:45 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: yynyygy, 16822

Eli Zaretskii <eliz@gnu.org> writes:

> I admit that I don't understand the problem.  Why should the display
> be different in these two cases?  It sounds like the (unspelled-out)
> assumption is that showing the region is more important than showing
> the (mis)matched parens?  But if this is the assumption, then I don't
> think I agree: show-paren-mode is for _temporary_ display of the
> parens, so if it temporarily obscures the region, it's perfectly okay.
>
> Adding a new face would bump into the problem of making sure the
> colors of this new face are always visible and distinguishable from
> the other two colors.  I don't see how that could work reliably.

I was thinking that perhaps the region-blink-face would tweak the
foreground (while the normal only tweaks the background), so by merging
we'd see some difference.

But you're right -- it's hard to see what the use case here is, really,
beyond "it'd be nice".  So I'm closing this bug report.

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





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

end of thread, other threads:[~2021-07-15  8:45 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-20  7:40 bug#16822: 24.3.50; show-paren-mode adds confusion to active region yynyygy
2014-02-20 17:44 ` Drew Adams
2014-02-20 21:43   ` Stefan Monnier
2021-07-15  4:54 ` Lars Ingebrigtsen
2021-07-15  4:59 ` Lars Ingebrigtsen
2021-07-15  6:53   ` Eli Zaretskii
2021-07-15  8:45     ` Lars Ingebrigtsen

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).