GNU bug report logs - #28621
Proposed patch for doc of posn-window and code of posn-set-point to handle frame arguments

Previous Next

Package: emacs;

Reported by: rswgnu <at> gmail.com

Date: Wed, 27 Sep 2017 16:03:02 UTC

Severity: minor

Tags: fixed, patch

Fixed in version 28.1

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

Bug is archived. No further changes may be made.

Full log


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

From: martin rudalics <rudalics <at> gmx.at>
To: rswgnu <at> gmail.com, Eli Zaretskii <eliz <at> gnu.org>
Cc: 28621 <at> debbugs.gnu.org
Subject: Re: bug#28621: Proposed patch for doc of posn-window and code of
 posn-set-point to handle frame arguments
Date: Sat, 30 Sep 2017 10:32:47 +0200
>>> !      (select-window (frame-selected-window (posn-window position))))
>>
>> Why should we select the selected-window on that frame in this case?
>>
>
> ​Because when position includes a window, the window is selected (the
> latter logic in this function).  So if position is in a window in another
> frame, shouldn't we select the window there?  We are trying to set point
> here
> based on a mouse position​, so the user must have moved his focus there.  I
> am
> looking for input on whether the logic is right.

(frame-selected-window (posn-window position)) might not be the window
under the mouse cursor.

martin





This bug report was last modified 4 years and 279 days ago.

Previous Next


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