Re: master 4fb028be6c 3/3: Fix releasing the mouse on top of the tool bar on MS Windows

[Eli Zaretskii]

> From: Po Lu <luangruo@yahoo.com>
> Cc: emacs-devel@gnu.org
> Date: Sat, 30 Apr 2022 15:50:53 +0800
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >>                      window = window_from_coordinates (f, x, y, 0, 1, 1);
> >>  
> >> -                    if (EQ (window, f->tool_bar_window))
> >> +                    if (EQ (window, f->tool_bar_window)
> >> +                  && (inev.modifiers & down_modifier
> >> +                      || f->last_tool_bar_item != -1))
> >>                        {
> >>                          w32_handle_tool_bar_click (f, &inev);
> >>                          tool_bar_p = 1;
> >> 
> >
> > Can you tell what was the problem, and how to see it happening before
> > the change?
> >
> > Thanks.
> 
> Begin dragging to select some text but release the mouse pointer on top
> of the toolbar.  Emacs will continue selecting text until you click
> somewhere else, because the button release event is consumed by the tool
> bar and never reaches Lisp.

Thanks.

For the future, please make a point of describing such situations in
the comments, they are invaluable when making changes in the relevant
code, especially if the change isn't related to any bug, so there's no
discussion to read and no recipes to try.