GNU bug report logs - #20398
25.0.50; Clicking `window-divider-last-pixel' in help for `window-divider'

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 20398 in the body.
You can then email your comments to 20398 AT debbugs.gnu.org in the normal way.

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

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

From: Drew Adams <drew.adams <at> oracle.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 25.0.50; Clicking `window-divider-last-pixel' in help for
 `window-divider'
Date: Tue, 21 Apr 2015 13:02:01 -0700 (PDT)

.. shows the error:
window-divider-first-pixel is void as a variable.

Help should show the doc string of the face, not the variable,


In GNU Emacs 25.0.50.1 (i686-pc-mingw32)
 of 2014-10-20 on LEG570
Bzr revision: 118168 rgm <at> gnu.org-20141020195941-icp42t8ttcnud09g
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking=yes,glyphs CPPFLAGS=-DGLYPH_DEBUG=1'

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Drew Adams <drew.adams <at> oracle.com>
Cc: 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Wed, 22 Apr 2015 11:08:56 +0300

> Date: Tue, 21 Apr 2015 13:02:01 -0700 (PDT)
> From: Drew Adams <drew.adams <at> oracle.com>
> 
> .. shows the error:
> window-divider-first-pixel is void as a variable.

Not here, it doesn't.  It's possible that this problem was already
fixed (you are using an ancient binary).  Or it's possible that the
recipe you used to trigger this is different from what I used (click
"Help->Describe->Describe Face", then type "window-divider RET" at the
prompt), in which case please show your recipe.

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

From: martin rudalics <rudalics <at> gmx.at>
To: Eli Zaretskii <eliz <at> gnu.org>, Drew Adams <drew.adams <at> oracle.com>
Cc: 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50; Clicking `window-divider-last-pixel' in help
 for `window-divider'
Date: Wed, 22 Apr 2015 11:34:16 +0200

>> .. shows the error:
>> window-divider-first-pixel is void as a variable.
>
> Not here, it doesn't.  It's possible that this problem was already
> fixed (you are using an ancient binary).  Or it's possible that the
> recipe you used to trigger this is different from what I used (click
> "Help->Describe->Describe Face", then type "window-divider RET" at the
> prompt), in which case please show your recipe.

It happens in customization buffers.  Here I can trigger it by

M-x customize-face RET window-divider RET

and mouse-clicking 'window-divider-first-pixel' in the doc-string.

martin

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 20398 <at> debbugs.gnu.org, drew.adams <at> oracle.com
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Wed, 22 Apr 2015 13:26:44 +0300

> Date: Wed, 22 Apr 2015 11:34:16 +0200
> From: martin rudalics <rudalics <at> gmx.at>
> CC: 20398 <at> debbugs.gnu.org
> 
> I can trigger it by
> 
> M-x customize-face RET window-divider RET
> 
> and mouse-clicking 'window-divider-first-pixel' in the doc-string.

That doesn't cause an error, just a message that it's not a variable.

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

From: Drew Adams <drew.adams <at> oracle.com>
To: Eli Zaretskii <eliz <at> gnu.org>, martin rudalics <rudalics <at> gmx.at>
Cc: 20398 <at> debbugs.gnu.org
Subject: RE: bug#20398: 25.0.50; Clicking `window-divider-last-pixel' in help
 for `window-divider'
Date: Wed, 22 Apr 2015 06:42:40 -0700 (PDT)

> > I can trigger it by
> > M-x customize-face RET window-divider RET
> > and mouse-clicking 'window-divider-first-pixel' in the doc-string.
> 
> That doesn't cause an error, just a message that it's not a
> variable.

The point is that from that customization buffer, for the related
face whose help mentions this face, you should be able to get
to this face.  A guess is that it might suffice to put the word
"face" before the face name in the doc string.  Just a wild guess.

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

From: Glenn Morris <rgm <at> gnu.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: martin rudalics <rudalics <at> gmx.at>, 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Wed, 22 Apr 2015 18:50:13 -0400

Eli Zaretskii wrote:

>> M-x customize-face RET window-divider RET
>> 
>> and mouse-clicking 'window-divider-first-pixel' in the doc-string.
>
> That doesn't cause an error, just a message that it's not a variable.

widget-documentation-link-action is surprisingly simplistic.
It doesn't even know that faces exist.
It should do what `help-make-xrefs' does, and use the presence of "face"
etc in the doc string as a cue. Ideally it would reuse the same code.

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

From: martin rudalics <rudalics <at> gmx.at>
To: Glenn Morris <rgm <at> gnu.org>, Eli Zaretskii <eliz <at> gnu.org>
Cc: 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50; Clicking `window-divider-last-pixel' in help
 for `window-divider'
Date: Thu, 23 Apr 2015 09:50:33 +0200

> widget-documentation-link-action is surprisingly simplistic.

I tried to make following links less simplistic in 557c7d6..62fe329 on
master.

> It doesn't even know that faces exist.
> It should do what `help-make-xrefs' does, and use the presence of "face"
> etc in the doc string as a cue. Ideally it would reuse the same code.

Ideally, yes.  Currently, I'm troubled by


Debugger entered--Lisp error: (error "No selection is available")
  signal(error ("No selection is available"))
  error("No selection is available")
  gui-get-primary-selection()
  mouse-yank-primary((mouse-2 (#<window 3 on *Customize Face: window-divider*> 615 (355 . 257) 4390875 nil 615 (42 . 14) nil (3 . 7) (8 . 16))))
  funcall-interactively(mouse-yank-primary (mouse-2 (#<window 3 on *Customize Face: window-divider*> 615 (355 . 257) 4390875 nil 615 (42 . 14) nil (3 . 7) (8 . 16))))
  call-interactively(mouse-yank-primary nil nil)
  command-execute(mouse-yank-primary)


with <mouse-1> on any such link in a customization buffer.  Can anyone
see this?  Does anyone have an idea where this mouse-2 yanking attempt
comes from?

martin

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: martin rudalics <rudalics <at> gmx.at>
Cc: rgm <at> gnu.org, 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Thu, 23 Apr 2015 13:37:16 +0300

> Date: Thu, 23 Apr 2015 09:50:33 +0200
> From: martin rudalics <rudalics <at> gmx.at>
> CC: 20398 <at> debbugs.gnu.org
> 
> I'm troubled by
> 
> 
> Debugger entered--Lisp error: (error "No selection is available")
>    signal(error ("No selection is available"))
>    error("No selection is available")
>    gui-get-primary-selection()
>    mouse-yank-primary((mouse-2 (#<window 3 on *Customize Face: window-divider*> 615 (355 . 257) 4390875 nil 615 (42 . 14) nil (3 . 7) (8 . 16))))
>    funcall-interactively(mouse-yank-primary (mouse-2 (#<window 3 on *Customize Face: window-divider*> 615 (355 . 257) 4390875 nil 615 (42 . 14) nil (3 . 7) (8 . 16))))
>    call-interactively(mouse-yank-primary nil nil)
>    command-execute(mouse-yank-primary)
> 
> 
> with <mouse-1> on any such link in a customization buffer.  Can anyone
> see this?

I see this.  Everyone should, AFAICS.

> Does anyone have an idea where this mouse-2 yanking attempt comes
> from?

It comes from the command loop.  I think mouse-1 remapping, as
implied by the return value of mouse-on-link-p, has something to do
with that.

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

From: martin rudalics <rudalics <at> gmx.at>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: rgm <at> gnu.org, 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50; Clicking `window-divider-last-pixel' in help
 for `window-divider'
Date: Sat, 02 May 2015 10:57:28 +0200

>> Debugger entered--Lisp error: (error "No selection is available")
>>     signal(error ("No selection is available"))
>>     error("No selection is available")
>>     gui-get-primary-selection()
>>     mouse-yank-primary((mouse-2 (#<window 3 on *Customize Face: window-divider*> 615 (355 . 257) 4390875 nil 615 (42 . 14) nil (3 . 7) (8 . 16))))
>>     funcall-interactively(mouse-yank-primary (mouse-2 (#<window 3 on *Customize Face: window-divider*> 615 (355 . 257) 4390875 nil 615 (42 . 14) nil (3 . 7) (8 . 16))))
>>     call-interactively(mouse-yank-primary nil nil)
>>     command-execute(mouse-yank-primary)
>>
>>
>> with <mouse-1> on any such link in a customization buffer.  Can anyone
>> see this?
>
> I see this.  Everyone should, AFAICS.
>
>> Does anyone have an idea where this mouse-2 yanking attempt comes
>> from?
>
> It comes from the command loop.  I think mouse-1 remapping, as
> implied by the return value of mouse-on-link-p, has something to do
> with that.

If someone just told me how to investigate this.  So far I failed
miserably to do that :-(

martin

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: martin rudalics <rudalics <at> gmx.at>
Cc: rgm <at> gnu.org, 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Sat, 02 May 2015 12:09:40 +0300

> Date: Sat, 02 May 2015 10:57:28 +0200
> From: martin rudalics <rudalics <at> gmx.at>
> CC: rgm <at> gnu.org, 20398 <at> debbugs.gnu.org
> 
>  >> Does anyone have an idea where this mouse-2 yanking attempt comes
>  >> from?
>  >
>  > It comes from the command loop.  I think mouse-1 remapping, as
>  > implied by the return value of mouse-on-link-p, has something to do
>  > with that.
> 
> If someone just told me how to investigate this.  So far I failed
> miserably to do that :-(

I cannot be that person: I also tried and failed.

Sorry.

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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 20398 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Sun, 03 May 2015 01:42:17 -0400

> If someone just told me how to investigate this.  So far I failed
> miserably to do that :-(

I'll have to try and investigate it, to see how/if I could help you with
it,


        Stefan

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

From: martin rudalics <rudalics <at> gmx.at>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 20398 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#20398: 25.0.50; Clicking `window-divider-last-pixel' in help
 for `window-divider'
Date: Sun, 03 May 2015 17:04:32 +0200

> I'll have to try and investigate it, to see how/if I could help you with
> it,

It's been already reported as Bug#19185.  The only solution I can think
of is the trivial one below.

martin

--- a/lisp/wid-edit.el
+++ b/lisp/wid-edit.el
@@ -854,6 +854,9 @@ button end points."
     (define-key map [backtab] 'widget-backward)
     (define-key map [down-mouse-2] 'widget-button-click)
     (define-key map [down-mouse-1] 'widget-button-click)
+    ;; Avoid `mouse-yank-primary' with `mouse-1-click-follows-link'
+    ;; non-nil (Bug#19185).
+    (define-key map [mouse-2] 'widget-button-click)
     ;; The following definition needs to avoid using escape sequences that
     ;; might get converted to ^M when building loaddefs.el
     (define-key map [(control ?m)] 'widget-button-press)

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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 20398 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Sun, 03 May 2015 21:56:02 -0400

>> I'll have to try and investigate it, to see how/if I could help you with it,
> It's been already reported as Bug#19185.  The only solution I can think
> of is the trivial one below.

Oh, yes I investigated already and IIRC the problem is in
widget-button-click: it's bound to the down-event but is really
a binding for the up event.

IIRC I didn't manage to fully understand the problem, but my impression
that widget-button-click should be split into two parts: one part bound
to the down-event (which would mostly do the "redraw the button as
pressed") and another bound to the up-event (which performs the actual
action associated with the click).

So I think the workaround you suggest is not that bad.  It's not a full
solution, but it does go in the right direction.


        Stefan

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 20398 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>,
 Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Sat, 30 Apr 2016 20:28:33 +0200

martin rudalics <rudalics <at> gmx.at> writes:

> It's been already reported as Bug#19185.  The only solution I can think
> of is the trivial one below.
>
> martin
>
> --- a/lisp/wid-edit.el
> +++ b/lisp/wid-edit.el
> @@ -854,6 +854,9 @@ button end points."
>      (define-key map [backtab] 'widget-backward)
>      (define-key map [down-mouse-2] 'widget-button-click)
>      (define-key map [down-mouse-1] 'widget-button-click)
> +    ;; Avoid `mouse-yank-primary' with `mouse-1-click-follows-link'
> +    ;; non-nil (Bug#19185).
> +    (define-key map [mouse-2] 'widget-button-click)
>      ;; The following definition needs to avoid using escape sequences that
>      ;; might get converted to ^M when building loaddefs.el

[...]

> So I think the workaround you suggest is not that bad.  It's not a full
> solution, but it does go in the right direction.

Martin, you never applied your patch.  Was there a reason for that?

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

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

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: martin rudalics <rudalics <at> gmx.at>, Eli Zaretskii <eliz <at> gnu.org>,
 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Sat, 30 Apr 2016 14:51:05 -0400

>>      (define-key map [backtab] 'widget-backward)
>>      (define-key map [down-mouse-2] 'widget-button-click)
>>      (define-key map [down-mouse-1] 'widget-button-click)
>> +    ;; Avoid `mouse-yank-primary' with `mouse-1-click-follows-link'
>> +    ;; non-nil (Bug#19185).
>> +    (define-key map [mouse-2] 'widget-button-click)

That seems to imply that widget-button-click will be run twice (once on
the down part and once on the up part).
How 'bout binding one of the two to `undefined' instead?


        Stefan

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

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: martin rudalics <rudalics <at> gmx.at>, Eli Zaretskii <eliz <at> gnu.org>,
 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Sat, 30 Apr 2016 15:27:22 -0400

>>> (define-key map [backtab] 'widget-backward)
>>> (define-key map [down-mouse-2] 'widget-button-click)
>>> (define-key map [down-mouse-1] 'widget-button-click)
>>> +    ;; Avoid `mouse-yank-primary' with `mouse-1-click-follows-link'
>>> +    ;; non-nil (Bug#19185).
>>> +    (define-key map [mouse-2] 'widget-button-click)

> That seems to imply that widget-button-click will be run twice (once on
> the down part and once on the up part).
> How 'bout binding one of the two to `undefined' instead?

Or just *move* the binding from [down-mouse-2] to [mouse-2] (since it's
a click binding and clicks should be bound to the up event rather than
the down event).


        Stefan

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

From: martin rudalics <rudalics <at> gmx.at>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>, 
 Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 20398 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#20398: 25.0.50; Clicking `window-divider-last-pixel' in help
 for `window-divider'
Date: Sun, 01 May 2016 10:23:50 +0200

>>>> (define-key map [backtab] 'widget-backward)
>>>> (define-key map [down-mouse-2] 'widget-button-click)
>>>> (define-key map [down-mouse-1] 'widget-button-click)
>>>> +    ;; Avoid `mouse-yank-primary' with `mouse-1-click-follows-link'
>>>> +    ;; non-nil (Bug#19185).
>>>> +    (define-key map [mouse-2] 'widget-button-click)
>
>> That seems to imply that widget-button-click will be run twice (once on
>> the down part and once on the up part).
>> How 'bout binding one of the two to `undefined' instead?
>
> Or just *move* the binding from [down-mouse-2] to [mouse-2] (since it's
> a click binding and clicks should be bound to the up event rather than
> the down event).

Shouldn't we then do the same here:

>>>> (define-key map [down-mouse-1] 'widget-button-click)

martin

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

From: martin rudalics <rudalics <at> gmx.at>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 20398 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>,
 Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: Re: bug#20398: 25.0.50; Clicking `window-divider-last-pixel' in help
 for `window-divider'
Date: Sun, 01 May 2016 10:23:44 +0200

>> So I think the workaround you suggest is not that bad.  It's not a full
>> solution, but it does go in the right direction.
>
> Martin, you never applied your patch.  Was there a reason for that?

It was a half-baked idea IIRC.  This issue requires more work, I'm
afraid.

martin

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

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 20398 <at> debbugs.gnu.org, Lars Ingebrigtsen <larsi <at> gnus.org>,
 Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Sun, 01 May 2016 09:15:47 -0400

> Shouldn't we then do the same here:
>>>>> (define-key map [down-mouse-1] 'widget-button-click)

I think so, yes.


        Stefan

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

From: martin rudalics <rudalics <at> gmx.at>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: 20398 <at> debbugs.gnu.org, Lars Ingebrigtsen <larsi <at> gnus.org>,
 Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#20398: 25.0.50; Clicking `window-divider-last-pixel' in help
 for `window-divider'
Date: Mon, 02 May 2016 10:00:17 +0200

>> Shouldn't we then do the same here:
>>>>>> (define-key map [down-mouse-1] 'widget-button-click)
>
> I think so, yes.

So the patch is as below.  Any objections to apply this to the emacs-25
branch?

martin

diff --git a/lisp/wid-edit.el b/lisp/wid-edit.el
index f0054be..0a0f458 100644
--- a/lisp/wid-edit.el
+++ b/lisp/wid-edit.el
@@ -852,8 +852,8 @@ widget-keymap
     (define-key map [(shift tab)] 'widget-backward)
     (put 'widget-backward :advertised-binding [(shift tab)])
     (define-key map [backtab] 'widget-backward)
-    (define-key map [down-mouse-2] 'widget-button-click)
-    (define-key map [down-mouse-1] 'widget-button-click)
+    (define-key map [mouse-2] 'widget-button-click)
+    (define-key map [mouse-1] 'widget-button-click)
     ;; The following definition needs to avoid using escape sequences that
     ;; might get converted to ^M when building loaddefs.el
     (define-key map [(control ?m)] 'widget-button-press)

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 20398 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>,
 Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Mon, 02 May 2016 19:13:53 +0200

martin rudalics <rudalics <at> gmx.at> writes:

> So the patch is as below.  Any objections to apply this to the emacs-25
> branch?

[...]

> -    (define-key map [down-mouse-2] 'widget-button-click)
> -    (define-key map [down-mouse-1] 'widget-button-click)
> +    (define-key map [mouse-2] 'widget-button-click)
> +    (define-key map [mouse-1] 'widget-button-click)

Isn't this a rather large change to go in at this point?  People may
have rebound `down-mouse-1' (etc) instead of `mouse-1'...

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

Message #72 received at 20398-done <at> debbugs.gnu.org (full text, mbox):

From: martin rudalics <rudalics <at> gmx.at>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 20398-done <at> debbugs.gnu.org,
 Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Subject: Re: bug#20398: 25.0.50; Clicking `window-divider-last-pixel' in help
 for `window-divider'
Date: Tue, 03 May 2016 08:44:38 +0200

>> So the patch is as below.  Any objections to apply this to the emacs-25
>> branch?
>
> [...]
>
>> -    (define-key map [down-mouse-2] 'widget-button-click)
>> -    (define-key map [down-mouse-1] 'widget-button-click)
>> +    (define-key map [mouse-2] 'widget-button-click)
>> +    (define-key map [mouse-1] 'widget-button-click)
>
> Isn't this a rather large change to go in at this point?  People may
> have rebound `down-mouse-1' (etc) instead of `mouse-1'...

No idea.  But counted as objection and therefore committed to master.
Closing the bug.

martin

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

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 20398 <at> debbugs.gnu.org
Subject: Re: bug#20398: 25.0.50;
 Clicking `window-divider-last-pixel' in help for `window-divider'
Date: Tue, 03 May 2016 12:42:06 -0400

> No idea.  But counted as objection and therefore committed to master.

FWIW, I also think it was too risky for emacs-25.

> Closing the bug.

Thanks,


        Stefan

Previous Next

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