On Wed, Dec 10, 2025 at 09:23:22PM +0100, Scott Kostyshak wrote:
> On Tue, Sep 23, 2025 at 08:36:48PM +0000, Jean-Marc Lasgouttes wrote:
> > commit ad80d13026a80f649aec1932a503ca9c44ad70de
> > Author: Jean-Marc Lasgouttes <[email protected]>
> > Date:   Sun Sep 21 00:07:44 2025 +0200
> > 
> >     Fix preview drawing after mouse selection is finished
> >     
> >     In order to fix #8951, commit 22030781 was backported from gadmm's
> >     lyx-unstable branch. What this does is record when one is selecting,
> >     and block the the switch to preview when the cursor tip leaves the
> >     previewable inset.
> >     
> >     The problem is that there is no notification given when the selection
> >     action is finished (mouse release) and therefore metrics are not
> >     recomputed. So the equation is correctly shown as a preview but relies
> >     on the old no-preview metrics.
> >     
> >     This commit implements such a notification:
> >     
> >     - Add virtual method notifyMouseSelectionDone() to Inset, InsetMathHull,
> >       InsetPreview and InsetIPA.
> >     
> >     - Add function notifyMouseSelectionDone() to Cursor.cpp.
> >     
> >     - In BufferView::mouseEventDispatch, call the function above when
> >       needed. Note that the code has been changed to preserve the cursor's
> >       anchor. There might be code that does not expect that.
> >     
> >     Fixes bug #13222.
> > ---
> 
> Starting with this commit, if I do a "quadruple click" (click fast four
> times in a row), the selection is drawn from the beginning of the line
> to the cursor.
> 
> Before this commit, a quadruple click would highlihgt from the cursor
> position to the end of the word.
> 
> I don't have a preference for which behavior there is for a quadruple
> click. I'm guessing we do not currently define what a quadruple click
> should do, so perhaps this is undefined behavior and isn't even a bug.
> Nonetheless, the change in behavior seemed unintended so I am reported
> it.

Time to file a bug in trac?

Pavel
-- 
lyx-devel mailing list
[email protected]
https://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to