GNU bug report logs - #23690
25.0.94; Window resizing regression

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 23690 in the body.
You can then email your comments to 23690 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: Stephen Berman <stephen.berman <at> gmx.net>
To: bug-gnu-emacs <at> gnu.org
Subject: 25.0.94; Window resizing regression
Date: Fri, 03 Jun 2016 14:23:11 +0200

0. emacs -Q
1. Type `C-x 3' to split the selected window into two side-by-side
   windows.
2. Sanity check: click mouse-1 on the divider between the mode lines,
   and observe that by dragging the mouse you can move the vertical
   boundary between the windows to the left or right.
3. In the menu Options->Show/Hide->Scroll-bar click the "On the Left"
   button to make the scroll bars appear on the left side of the
   windows.
4. Repeat step 2.
=> The vertical boundary between the windows does not change, but
   instead the echo area displays the message "No window on the right of
   this one" (although the selected window is the one on the left).

This is a regression from Emacs 24.5, where step 4 works just like step
2.  Git bisect identifies the following as the responsible commit:

commit 3110159d1fb6b6071803355951d9ae314095df36
Author: Martin Rudalics <rudalics <at> gmx.at>
Date:   Fri Oct 24 11:58:43 2014 +0200

    Improve mouse dragging of frame edges.
    
    * keyboard.c (make_lispy_position): Return coordinates also when
    on scroll bars, fringes, margins or not in a window.
    * xdisp.c (show_mouse_face): Don't change cursor face during
    mouse tracking.
    * mouse.el (mouse-drag-line): Don't use mouse-pixel-position.
    Calculate increment from last position instead of window edge.
    Add right- and bottom-divider bindings to transient map.



In GNU Emacs 25.0.94.1 (x86_64-suse-linux-gnu, GTK+ Version 3.14.15)
 of 2016-06-03 built on rosalinde
Repository revision: 9c28e70b4b4b1887030e41103b5f6dd8304585e6
Windowing system distributor 'The X.Org Foundation', version 11.0.11601000
System Description:	openSUSE 13.2 (Harlequin) (x86_64)

Configured using:
 'configure --with-xwidgets 'CFLAGS=-Og -g3''

Configured features:
XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND DBUS GCONF GSETTINGS NOTIFY
GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB TOOLKIT_SCROLL_BARS
GTK3 X11 XWIDGETS

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

From: martin rudalics <rudalics <at> gmx.at>
To: Stephen Berman <stephen.berman <at> gmx.net>, 23690 <at> debbugs.gnu.org
Subject: Re: bug#23690: 25.0.94; Window resizing regression
Date: Fri, 03 Jun 2016 15:54:11 +0200

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

> 0. emacs -Q
> 1. Type `C-x 3' to split the selected window into two side-by-side
>     windows.
> 2. Sanity check: click mouse-1 on the divider between the mode lines,
>     and observe that by dragging the mouse you can move the vertical
>     boundary between the windows to the left or right.
> 3. In the menu Options->Show/Hide->Scroll-bar click the "On the Left"
>     button to make the scroll bars appear on the left side of the
>     windows.
> 4. Repeat step 2.
> => The vertical boundary between the windows does not change, but
>     instead the echo area displays the message "No window on the right of
>     this one" (although the selected window is the one on the left).
>
> This is a regression from Emacs 24.5, where step 4 works just like step
> 2.  Git bisect identifies the following as the responsible commit:
>
> commit 3110159d1fb6b6071803355951d9ae314095df36
> Author: Martin Rudalics <rudalics <at> gmx.at>
> Date:   Fri Oct 24 11:58:43 2014 +0200
>
>      Improve mouse dragging of frame edges.
>
>      * keyboard.c (make_lispy_position): Return coordinates also when
>      on scroll bars, fringes, margins or not in a window.
>      * xdisp.c (show_mouse_face): Don't change cursor face during
>      mouse tracking.
>      * mouse.el (mouse-drag-line): Don't use mouse-pixel-position.
>      Calculate increment from last position instead of window edge.
>      Add right- and bottom-divider bindings to transient map.

Would the attached patch fix it for you?

Thanks, martin

[mouse.diff (text/plain, attachment)]

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

From: Stephen Berman <stephen.berman <at> gmx.net>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 23690 <at> debbugs.gnu.org
Subject: Re: bug#23690: 25.0.94; Window resizing regression
Date: Fri, 03 Jun 2016 16:30:49 +0200

On Fri, 03 Jun 2016 15:54:11 +0200 martin rudalics <rudalics <at> gmx.at> wrote:

>> 0. emacs -Q
>> 1. Type `C-x 3' to split the selected window into two side-by-side
>>     windows.
>> 2. Sanity check: click mouse-1 on the divider between the mode lines,
>>     and observe that by dragging the mouse you can move the vertical
>>     boundary between the windows to the left or right.
>> 3. In the menu Options->Show/Hide->Scroll-bar click the "On the Left"
>>     button to make the scroll bars appear on the left side of the
>>     windows.
>> 4. Repeat step 2.
>> => The vertical boundary between the windows does not change, but
>>     instead the echo area displays the message "No window on the right of
>>     this one" (although the selected window is the one on the left).
>>
>> This is a regression from Emacs 24.5, where step 4 works just like step
>> 2.  Git bisect identifies the following as the responsible commit:
>>
>> commit 3110159d1fb6b6071803355951d9ae314095df36
>> Author: Martin Rudalics <rudalics <at> gmx.at>
>> Date:   Fri Oct 24 11:58:43 2014 +0200
>>
>>      Improve mouse dragging of frame edges.
>>
>>      * keyboard.c (make_lispy_position): Return coordinates also when
>>      on scroll bars, fringes, margins or not in a window.
>>      * xdisp.c (show_mouse_face): Don't change cursor face during
>>      mouse tracking.
>>      * mouse.el (mouse-drag-line): Don't use mouse-pixel-position.
>>      Calculate increment from last position instead of window edge.
>>      Add right- and bottom-divider bindings to transient map.
>
> Would the attached patch fix it for you?

It does indeed; thanks!

Steve Berman

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

From: martin rudalics <rudalics <at> gmx.at>
To: Stephen Berman <stephen.berman <at> gmx.net>
Cc: 23690 <at> debbugs.gnu.org
Subject: Re: bug#23690: 25.0.94; Window resizing regression
Date: Sat, 04 Jun 2016 11:48:24 +0200

>> Would the attached patch fix it for you?
>
> It does indeed; thanks!

I'd very much like to push this to the release branch.  As it stands,
the behavior of this is botched on Lucid and Motif which both put the
scroll bar on left by default.  Eli?

Thanks for testing, martin

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 23690 <at> debbugs.gnu.org, stephen.berman <at> gmx.net
Subject: Re: bug#23690: 25.0.94; Window resizing regression
Date: Sat, 04 Jun 2016 13:53:47 +0300

> Date: Sat, 04 Jun 2016 11:48:24 +0200
> From: martin rudalics <rudalics <at> gmx.at>
> Cc: 23690 <at> debbugs.gnu.org
> 
>  >> Would the attached patch fix it for you?
>  >
>  > It does indeed; thanks!
> 
> I'd very much like to push this to the release branch.  As it stands,
> the behavior of this is botched on Lucid and Motif which both put the
> scroll bar on left by default.  Eli?

You'll have to convince me.  It sounds like the problem is triggered
by a combination of 2 unlikely events, so I'm unsure why we should
risk last-minute problems.

Thanks.

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

From: Stephen Berman <stephen.berman <at> gmx.net>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: martin rudalics <rudalics <at> gmx.at>, 23690 <at> debbugs.gnu.org
Subject: Re: bug#23690: 25.0.94; Window resizing regression
Date: Sat, 04 Jun 2016 13:12:29 +0200

On Sat, 04 Jun 2016 13:53:47 +0300 Eli Zaretskii <eliz <at> gnu.org> wrote:

>> Date: Sat, 04 Jun 2016 11:48:24 +0200
>> From: martin rudalics <rudalics <at> gmx.at>
>> Cc: 23690 <at> debbugs.gnu.org
>> 
>>  >> Would the attached patch fix it for you?
>>  >
>>  > It does indeed; thanks!
>> 
>> I'd very much like to push this to the release branch.  As it stands,
>> the behavior of this is botched on Lucid and Motif which both put the
>> scroll bar on left by default.  Eli?
>
> You'll have to convince me.  It sounds like the problem is triggered
> by a combination of 2 unlikely events, so I'm unsure why we should
> risk last-minute problems.

It is a regression from 24.5 (and earlier).  It is also documented in
(emacs)Mode Line Mouse: "Furthermore, by clicking and dragging ‘mouse-1’
on the divider between two side-by-side mode lines, you can move the
vertical boundary to the left or right."  On the other hand there's also
window-divider-mode, which is new in 25.1 provides the full behavior
that is currently missing due to this bug; however, it is not enabled by
default.  But if the fix is safe (it seems so to me, but I'm not
familiar enough with the code to be certain), I would like it in 25.1.
(I use the Gtk+ toolkit but also have the scroll bar on the left).

Steve Berman

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

From: martin rudalics <rudalics <at> gmx.at>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 23690 <at> debbugs.gnu.org, stephen.berman <at> gmx.net
Subject: Re: bug#23690: 25.0.94; Window resizing regression
Date: Sat, 04 Jun 2016 15:07:42 +0200

> You'll have to convince me.  It sounds like the problem is triggered
> by a combination of 2 unlikely events,

When you build with Lucid, Motif or without toolkit support, the scroll
bars appear by default on the left.  After C-x 3 you cannot use the
mouse to resize the windows.  No unlikely event here.  This is a clear
regression.

> so I'm unsure why we should
> risk last-minute problems.

The fix is governed by two conditions - that there are no window
dividers and that the scroll bars are on the left.  Everbody else should
be unaffected.  How much time do we have left until the last minute?

martin

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 23690 <at> debbugs.gnu.org, stephen.berman <at> gmx.net
Subject: Re: bug#23690: 25.0.94; Window resizing regression
Date: Sat, 04 Jun 2016 18:05:30 +0300

> Date: Sat, 04 Jun 2016 15:07:42 +0200
> From: martin rudalics <rudalics <at> gmx.at>
> CC: stephen.berman <at> gmx.net, 23690 <at> debbugs.gnu.org
> 
>  > You'll have to convince me.  It sounds like the problem is triggered
>  > by a combination of 2 unlikely events,
> 
> When you build with Lucid, Motif or without toolkit support, the scroll
> bars appear by default on the left.  After C-x 3 you cannot use the
> mouse to resize the windows.  No unlikely event here.  This is a clear
> regression.

OK, let's push to emacs-25.

Thanks.

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

From: martin rudalics <rudalics <at> gmx.at>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: stephen.berman <at> gmx.net, 23690-done <at> debbugs.gnu.org
Subject: Re: bug#23690: 25.0.94; Window resizing regression
Date: Sun, 05 Jun 2016 11:53:17 +0200

> OK, let's push to emacs-25.

Done.  Closing this bug.

Thanks, martin

Previous Next

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