From: Stephen Berman <stephen.berman@gmx.net>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 23571@debbugs.gnu.org
Subject: bug#23571: 25.1.50; Click-mouse-1 = Drag-mouse-1 in Gnus Article buffer
Date: Tue, 07 Jun 2016 11:05:29 +0200 [thread overview]
Message-ID: <8760tlqsh2.fsf@gmx.net> (raw)
In-Reply-To: <83twh5spa1.fsf@gnu.org> (Eli Zaretskii's message of "Tue, 07 Jun 2016 05:31:34 +0300")
On Tue, 07 Jun 2016 05:31:34 +0300 Eli Zaretskii <eliz@gnu.org> wrote:
>> From: Stephen Berman <stephen.berman@gmx.net>
>> Cc: 23571@debbugs.gnu.org
>> Date: Mon, 06 Jun 2016 21:38:02 +0200
>>
>> >> 72166f2f3dba18f1217c666574032f5a0351ed65 is the first bad commit
>> >> commit 72166f2f3dba18f1217c666574032f5a0351ed65
>> >> Author: Martin Rudalics <rudalics@gmx.at>
>> >> Date: Tue May 3 08:38:49 2016 +0200
>> >>
>> >> Bind `widget-button-click' to mouse-1/-2 instead of down-mouse-1/-2
>> >>
>> >> * lisp/wid-edit.el (widget-keymap): Bind `widget-button-click'
>> >> to mouse-1/-2 instead of down-mouse-1/-2. Suggested by Stefan
>> >> Monnier. (Bug#19185, Bug#20398)
>> >
>> > So I guess Gnus needs to do something to countermand the low-level
>> > change, right?
>>
>> It turns out it's not just Gnus Article buffers (as presciently
>> suggested by the title of bug#23653, which I merged with this one): in
>> fact, the same problem appears to happen in all packages in which
>> buffers use widget-keymap; there are quite a few of these, as rgrepping
>> for widget-keymap on the lisp directory shows, and in all that I tried
>> (cus-edit, wid-browse, recentf, printing, secrets, image-dired,
>> todo-mode) the problem with mouse-1 occurred. So I think the fix should
>> be in widget-button-click, or somewhere at that level, and not in all of
>> its users.
>
> How do you do that without reintroducing the bug referenced in the
> above commit's log message?
I went through those two bug reports and I cannot reproduce the
problematic cases (bug#19185: mouse-2 running mouse-yank-primary in a
Gnus Article buffer; bug#20398: mouse-1 raising error "No selection is
available" on links in a Customize buffer) with -Q in current emacs-25,
which does not include the above commit. And when I revert that commit
in master I also cannot reproduce those problems. So it's not clear to
me what that commit is trying to fix.
Steve Berman
next prev parent reply other threads:[~2016-06-07 9:05 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-18 13:27 bug#23571: 25.1.50; Click-mouse-1 = Drag-mouse-1 in Gnus Article buffer Stephen Berman
2016-05-26 16:09 ` Stephen Berman
2016-05-26 16:37 ` Eli Zaretskii
2016-05-27 9:20 ` Stephen Berman
2016-06-06 19:38 ` Stephen Berman
2016-06-07 2:31 ` Eli Zaretskii
2016-06-07 9:05 ` Stephen Berman [this message]
2016-06-07 15:03 ` Eli Zaretskii
2016-06-08 6:34 ` martin rudalics
2016-06-08 16:42 ` Eli Zaretskii
2016-06-09 8:39 ` martin rudalics
2016-06-09 9:00 ` Stephen Berman
2016-06-10 7:15 ` martin rudalics
2018-04-12 16:04 ` Lars Ingebrigtsen
2016-06-10 17:18 ` Stefan Monnier
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=8760tlqsh2.fsf@gmx.net \
--to=stephen.berman@gmx.net \
--cc=23571@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 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.