Advanced find changes of behavior in master

Scott Kostyshak skostysh at lyx.org
Tue Mar 30 14:34:24 UTC 2021


On Mon, Mar 29, 2021 at 11:11:33AM +0200, Jean-Marc Lasgouttes wrote:
> Le 25/11/2020 à 15:35, Scott Kostyshak a écrit :
> > I see a couple of changes in behavior in advanced find that can be
> > reproduced with the Welcome.lyx file (I attached it for convenience).
> > 
> > 1. In advanced find, put "a" in the find area and "c" in the replace
> >     area and do "Replace all". It takes about 10 seconds with 2.3.0, and
> >     about 25 seconds with master. I remember when testing before that
> >     advanced find was much faster than 2.3.0, so if this is just a weird
> >     "worst case" scenario, then I think that's fine.
> > 
> > 2. In 2.3.0, there are more workarea updates. This way, the user can see
> >     what is being replaced and where. In master, there is only one work
> >     area update at the end. This seems like an improvement in theory, but
> >     since the advanced find takes a while, it is kind of nice to watch it
> >     so that we know it is not stuck and can get an idea of the progress.
> >     In master, it is strange to see toolbar buttons being enabled and
> >     disabled (based on where the invisible cursor is) and the scrollbar
> >     moving, but not to see the cursor moving and the work area updates. I
> >     don't have a preference on this. I just mention it in case the change
> >     was not intended.
> 
> What are the issues that remain here?

If I remember correctly, (1) is because of the multiple calls to update
layouts; and (2) I think was improved (by you I think) so that the
toolbar icons do not change. The workarea is not updated so there is a
difference, but one can follow the progress by watching the scrollbar
move.

Scott
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.lyx.org/pipermail/lyx-devel/attachments/20210330/454c5a3a/attachment.asc>


More information about the lyx-devel mailing list