View Single Post
Old 04-14-2021, 03:43 PM   #71
KevinH
Sigil Developer
KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.
 
Posts: 8,893
Karma: 6120478
Join Date: Nov 2009
Device: many
Hmm ... do you see the double-pump with Sigil 1.4.3 and Preview as well. That is when we introduced the URLSchemeHandler. I wonder if that is the change.

The double-pump is not actually a double-pump but appears as one. Search first opens the target file and then in all versions less than master, it will wait for Preview to load and scroll to its previously stored position, when that completes, CV then moves the cursor to the found text in that file and highlights it. Preview should then happily move to that new location without needing a full reload, Preview should just need to scroll.

But the video clearly shows Preview doing a full reload and then scroll.

Nothing directly changed in any of this code between 1.3.0 to 1.5.1 and this does not happen in macOS (or if it does it happens so fast you can not see it).

If this is not related to the addition of the URLScheme handler after 1.3.0, then we need to look at the signals being emitted by CodeView after its initial load while the scroll to the target text is being done. Somehow that is triggering a complete reload of Preview when only a scroll is done.

Can you see the same thing on Linux?
KevinH is offline   Reply With Quote