GNU bug report logs - #56421
[PATCH] Make mouse-2 respect delete-selection mode

Previous Next

To reply to this bug, email your comments to 56421 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Visuwesh <visuweshm <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] Make mouse-2 respect delete-selection mode
Date: Wed, 06 Jul 2022 20:28:48 +0530

[Message part 1 (text/plain, inline)]

Tags: patch

Yanking with the mouse (i.e., mouse-2) does not replace the selected
region if delete-selection-mode is turned on.  Attached patch makes the
mouse yank commands delete-selection-mode aware.

[0001-Make-mouse-3-respect-delete-selection-mode.patch (text/patch, attachment)]

[Message part 3 (text/plain, inline)]

In GNU Emacs 29.0.50 (build 3, x86_64-pc-linux-gnu, X toolkit, Xaw scroll bars)
 of 2022-07-03 built on astatine
Repository revision: c4930e938b6f9f21b4a9c37912d2b6ce4796b9a2
Repository branch: tamil-phonetic2
Windowing system distributor 'The X.Org Foundation', version 11.0.12101003
System Description: Debian GNU/Linux 11 (bullseye)

Configured using:
 'configure --with-modules --with-sound=alsa --with-x-toolkit=lucid
 --with-json --without-xaw3d --without-gconf --without-libsystemd
 --with-x --without-cairo'

Message #10 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Visuwesh <visuweshm <at> gmail.com>
Cc: 56421 <at> debbugs.gnu.org
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Wed, 6 Jul 2022 17:11:35 +0200

severity 56421 wishlist
thanks

Visuwesh <visuweshm <at> gmail.com> writes:

> Yanking with the mouse (i.e., mouse-2) does not replace the selected
> region if delete-selection-mode is turned on.  Attached patch makes the
> mouse yank commands delete-selection-mode aware.

What does other software in X do here?  From some light testing, it
seems to me like it does not replace the selection?

Message #13 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Visuwesh <visuweshm <at> gmail.com>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: 56421 <at> debbugs.gnu.org
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Wed, 06 Jul 2022 20:46:17 +0530

[புதன் ஜூலை 06, 2022] Stefan Kangas wrote:

> Visuwesh <visuweshm <at> gmail.com> writes:
>
>> Yanking with the mouse (i.e., mouse-2) does not replace the selected
>> region if delete-selection-mode is turned on.  Attached patch makes the
>> mouse yank commands delete-selection-mode aware.
>
> What does other software in X do here?  From some light testing, it
> seems to me like it does not replace the selection?

Ah, I did not notice that it is done by Chrome either, and I don't have
any other X app handy to test.  I was surprised when mouse-2 did not
replace the region, and found it inconsistent.

Message #16 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Visuwesh <visuweshm <at> gmail.com>
Cc: 56421 <at> debbugs.gnu.org
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Wed, 6 Jul 2022 17:58:24 +0200

Visuwesh <visuweshm <at> gmail.com> writes:

> Ah, I did not notice that it is done by Chrome either, and I don't have
> any other X app handy to test.  I was surprised when mouse-2 did not
> replace the region, and found it inconsistent.

So I guess the new behaviour should be optional?

Message #19 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Visuwesh <visuweshm <at> gmail.com>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: 56421 <at> debbugs.gnu.org
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Wed, 06 Jul 2022 21:31:01 +0530

[புதன் ஜூலை 06, 2022] Stefan Kangas wrote:

> Visuwesh <visuweshm <at> gmail.com> writes:
>
>> Ah, I did not notice that it is done by Chrome either, and I don't have
>> any other X app handy to test.  I was surprised when mouse-2 did not
>> replace the region, and found it inconsistent.
>
> So I guess the new behaviour should be optional?

If you think we should be consistent, then I'd rather drop the patch
than add yet another defcustom.  It isn't hard to copy those lines to my
config anyway.

Message #22 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Visuwesh <visuweshm <at> gmail.com>
Cc: 56421 <at> debbugs.gnu.org
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Wed, 6 Jul 2022 18:14:06 +0200

tags 56421 + moreinfo
thanks

Visuwesh <visuweshm <at> gmail.com> writes:

> If you think we should be consistent, then I'd rather drop the patch
> than add yet another defcustom.  It isn't hard to copy those lines to my
> config anyway.

Let's see if anyone else has an opinion here.

Message #27 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Juri Linkov <juri <at> linkov.net>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: 56421 <at> debbugs.gnu.org, Visuwesh <visuweshm <at> gmail.com>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Wed, 06 Jul 2022 20:57:11 +0300

>> If you think we should be consistent, then I'd rather drop the patch
>> than add yet another defcustom.  It isn't hard to copy those lines to my
>> config anyway.
>
> Let's see if anyone else has an opinion here.

This should be the right thing to do, some time ago
we already discussed this because I had these uncommitted lines:

+(put 'mouse-yank-primary 'delete-selection 'yank)
+(put 'mouse-yank-at-click 'delete-selection 'yank)
+(put 'menu-bar-select-yank 'delete-selection 'yank)

BTW, I also had this line

+(put 'just-one-space 'delete-selection t)

but since M-SPC is bound to cycle-spacing now,
I'm not sure about M-SPC anymore.  How does
cycle-spacing handle regions?  Maybe it should
squeeze all consequent spaces to one space
everywhere in the region?

Message #30 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Visuwesh <visuweshm <at> gmail.com>
Cc: 56421 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Thu, 07 Jul 2022 09:12:28 +0800

Visuwesh <visuweshm <at> gmail.com> writes:

> Ah, I did not notice that it is done by Chrome either, and I don't have
> any other X app handy to test.  I was surprised when mouse-2 did not
> replace the region, and found it inconsistent.

Having Button2 replace the region is not the right thing on X, which is
actually to deactivate the mark upon losing ownership of the primary
selection.  So what you talk about cannot happen in other X programs
(Emacs is the odd one out here.)

Message #33 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Po Lu <luangruo <at> yahoo.com>
Cc: 56421 <at> debbugs.gnu.org, Visuwesh <visuweshm <at> gmail.com>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Thu, 7 Jul 2022 03:17:16 +0200

Po Lu <luangruo <at> yahoo.com> writes:

> Having Button2 replace the region is not the right thing on X, which is
> actually to deactivate the mark upon losing ownership of the primary
> selection.  So what you talk about cannot happen in other X programs
> (Emacs is the odd one out here.)

Are you saying that Emacs currently behaves differently from other
programs on X in this regard?

Message #36 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: 56421 <at> debbugs.gnu.org, Visuwesh <visuweshm <at> gmail.com>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Thu, 07 Jul 2022 10:57:21 +0800

Stefan Kangas <stefan <at> marxist.se> writes:

> Are you saying that Emacs currently behaves differently from other
> programs on X in this regard?

Yes, and it has been doing that for a very long time.

Message #39 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Visuwesh <visuweshm <at> gmail.com>
To: Juri Linkov <juri <at> linkov.net>
Cc: 56421 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Thu, 07 Jul 2022 09:35:46 +0530

[புதன் ஜூலை 06, 2022] Juri Linkov wrote:

>>> If you think we should be consistent, then I'd rather drop the patch
>>> than add yet another defcustom.  It isn't hard to copy those lines to my
>>> config anyway.
>>
>> Let's see if anyone else has an opinion here.
>
> This should be the right thing to do, some time ago
> we already discussed this because I had these uncommitted lines:

It is really convenient to follow up with a mouse-2 after a multi-click
mouse-1 to replace the region IMO.  No need to take your hands off the
mouse.

Message #42 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Juri Linkov <juri <at> linkov.net>
To: Visuwesh <visuweshm <at> gmail.com>
Cc: 56421 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Thu, 07 Jul 2022 09:23:31 +0300

>>>> If you think we should be consistent, then I'd rather drop the patch
>>>> than add yet another defcustom.  It isn't hard to copy those lines to my
>>>> config anyway.
>>>
>>> Let's see if anyone else has an opinion here.
>>
>> This should be the right thing to do, some time ago
>> we already discussed this because I had these uncommitted lines:
>
> It is really convenient to follow up with a mouse-2 after a multi-click
> mouse-1 to replace the region IMO.  No need to take your hands off the
> mouse.

Do you have additional customization that allows you to achieve this
convenient behavior?  Such as setting select-active-regions to nil.

Message #45 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Visuwesh <visuweshm <at> gmail.com>
To: Juri Linkov <juri <at> linkov.net>
Cc: 56421 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Thu, 07 Jul 2022 12:17:06 +0530

[வியாழன் ஜூலை 07, 2022] Juri Linkov wrote:

>> It is really convenient to follow up with a mouse-2 after a multi-click
>> mouse-1 to replace the region IMO.  No need to take your hands off the
>> mouse.
>
> Do you have additional customization that allows you to achieve this
> convenient behavior?  Such as setting select-active-regions to nil.

I can't think of any that is particularly special about my
customisation, but I have delete-selection-temporary-region, and
mouse-yank-at-point set to t.  I also have select-enable-clipboard set
to nil since I like to separate Emacs' kill-ring and the Xorg clipboard
but I don't think that is relevant here.

Message #48 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Juri Linkov <juri <at> linkov.net>
Cc: 56421 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>,
 Visuwesh <visuweshm <at> gmail.com>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Mon, 11 Jul 2022 12:58:28 +0200

Juri Linkov <juri <at> linkov.net> writes:

>>> If you think we should be consistent, then I'd rather drop the patch
>>> than add yet another defcustom.  It isn't hard to copy those lines to my
>>> config anyway.
>>
>> Let's see if anyone else has an opinion here.
>
> This should be the right thing to do, some time ago
> we already discussed this because I had these uncommitted lines:
>
> +(put 'mouse-yank-primary 'delete-selection 'yank)
> +(put 'mouse-yank-at-click 'delete-selection 'yank)
> +(put 'menu-bar-select-yank 'delete-selection 'yank)

I agree, so I've applied Visuwesh's patch to Emacs 29.

> BTW, I also had this line
>
> +(put 'just-one-space 'delete-selection t)
>
> but since M-SPC is bound to cycle-spacing now,
> I'm not sure about M-SPC anymore.  How does
> cycle-spacing handle regions?  Maybe it should
> squeeze all consequent spaces to one space
> everywhere in the region?

Hm, not sure either.

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

Message #53 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: Visuwesh <visuweshm <at> gmail.com>, 56421 <at> debbugs.gnu.org,
 Stefan Kangas <stefan <at> marxist.se>, Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Mon, 11 Jul 2022 19:04:33 +0800

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> I agree, so I've applied Visuwesh's patch to Emacs 29.

This should be announced in NEWS as an incompatible editing change,
along with instructions to disable it.

Though IMNSHO, the right solution is to enable `lost-selection-mode' and
to ignore middle-clicks on the region, so this situation cannot happen
in the first place.

Message #56 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Po Lu <luangruo <at> yahoo.com>
Cc: Visuwesh <visuweshm <at> gmail.com>, 56421 <at> debbugs.gnu.org,
 Stefan Kangas <stefan <at> marxist.se>, Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Mon, 11 Jul 2022 13:10:00 +0200

Po Lu <luangruo <at> yahoo.com> writes:

> This should be announced in NEWS as an incompatible editing change,
> along with instructions to disable it.

It doesn't really feel NEWS-worthy to me -- it's a pretty natural DWIM
tweak.

> Though IMNSHO, the right solution is to enable `lost-selection-mode' and
> to ignore middle-clicks on the region, so this situation cannot happen
> in the first place.

Well, if you have `lost-selection-mode' switched on, it's even less
NEWS-worthy.  😀

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

Message #59 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: Visuwesh <visuweshm <at> gmail.com>, 56421 <at> debbugs.gnu.org,
 Stefan Kangas <stefan <at> marxist.se>, Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Mon, 11 Jul 2022 19:13:45 +0800

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> It doesn't really feel NEWS-worthy to me -- it's a pretty natural DWIM
> tweak.

I disagree, but how you or I feel isn't what should be guiding the
decision of whether or not to put something in NEWS.  It's a change to
very old behavior, and is thus newsworthy.

Message #62 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: Po Lu <luangruo <at> yahoo.com>, Juri Linkov <juri <at> linkov.net>,
 56421 <at> debbugs.gnu.org, Visuwesh <visuweshm <at> gmail.com>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Mon, 11 Jul 2022 04:37:30 -0700

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> It doesn't really feel NEWS-worthy to me -- it's a pretty natural DWIM
> tweak.

But it is different from the convention on X, right?

Message #65 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: Po Lu <luangruo <at> yahoo.com>, Juri Linkov <juri <at> linkov.net>,
 56421 <at> debbugs.gnu.org, Visuwesh <visuweshm <at> gmail.com>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Mon, 11 Jul 2022 13:39:09 +0200

Stefan Kangas <stefan <at> marxist.se> writes:

> But it is different from the convention on X, right?

Yes, but it only has an effect if Emacs doesn't heed the X convention
(i.e., lost-selection-mode isn't on).

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

Message #68 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: Juri Linkov <juri <at> linkov.net>, Lars Ingebrigtsen <larsi <at> gnus.org>,
 56421 <at> debbugs.gnu.org, Visuwesh <visuweshm <at> gmail.com>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Mon, 11 Jul 2022 19:39:59 +0800

Stefan Kangas <stefan <at> marxist.se> writes:

> But it is different from the convention on X, right?

Right.  That convention is what you would get with `lost-selection-mode'
(which I recently added) enabled, along with this change.

Message #75 received at 56421 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: Po Lu <luangruo <at> yahoo.com>, 56421 <at> debbugs.gnu.org,
 Visuwesh <visuweshm <at> gmail.com>, Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#56421: [PATCH] Make mouse-2 respect delete-selection mode
Date: Wed, 13 Jul 2022 13:39:12 +0200

I've now reopened this due to bug#56530 (and reverted the commit).


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

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.