Sigil-1.7.0 Released - Page 4

KevinH · 08-02-2021, 03:08 PM

Moving the setZoomFactor to the loadProgress slot allowed me to shrink the needed delay down to 30ms and still consistently see the syncing work.

So it helps but we still need the delay.

Pushed this to master since it is an improvement.

DiapDealer · 08-02-2021, 05:30 PM

Verified that the fix is working (at 30ms) on my overpowered Linux machine as well as my under-provisioned Windows 10 VM. Let's hope this is sufficient!

KevinH · 08-02-2021, 05:42 PM

Whew! That is good to know as we were rapidly running out of options.

Quote:

Originally Posted by DiapDealer

Verified that the fix is working (at 30ms) on my overpowered Linux machine as well as my under-provisioned Windows 10 VM. Let's hope this is sufficient!

DiapDealer · 08-02-2021, 06:12 PM

That's Qt5.15.2 on my Arch Linux machine, by the way. I confirmed the bug still existed before testing the fix. So it's clearly a long-standing problem.

Ashjuk · 08-03-2021, 03:21 AM

Quote:

Originally Posted by DiapDealer

It depends on the length of the file and where you are IN it as well. If the edits you're making are close enough to the top (or bottom) of a relatively small file, it's possible for the recent edit to still remain visible in the Preview Window even after the incorrect sync attempt. Try a good long xhtml file and scroll to the middle of it and see if that makes a difference.

I can see in the original images you posted that you're scrolled nearly to the bottom of the Preview Window. And that although the sync is still being botched after your edit, it's just not moving the needle enough to make a huge difference.

I have to admit I was using the test epub as supplied by Turtle. I will try the same process with a longer file to see what happens.

Quote:

Originally Posted by BeckyEbook

@Ashjuk: So how did you take your screenshots when we see Zoom 80% there? This can only be seen after clicking in the Preview window.
If you click on the Preview window, Code View synchronization will occur, and the problem we are discussing is Code View reverse synchronization -> Preview Window.

If you really have Code View 100% and Preview 80% then click on the red paragraph in the Code View and in the Preview window it will not be in the center, but a little above the center.

You are right, the edit point is a little above centre having inserted some text.

It seems I am getting confused because I was using the test epub which has relatively short files so the edit point does not move off screen.

Anyway you all appear to have fixed the bug so you should probably just ignore my ramblings.

KevinH · 08-03-2021, 09:57 AM

It is a timing thing so on some systems a delay of 0 (what Sigil 1.7 ships with) may actually work. On my system, that only works when zoom is 100%.

We could in fact use the current zoom value to decide how long or even if to delay. I will look into that.

Thanks for testing and reporting back.

Quote:

Originally Posted by Ashjuk

I have to admit I was using the test epub as supplied by Turtle. I will try the same process with a longer file to see what happens.

You are right, the edit point is a little above centre having inserted some text.

It seems I am getting confused because I was using the test epub which has relatively short files so the edit point does not move off screen.

Anyway you all appear to have fixed the bug so you should probably just ignore my ramblings.

Turtle91 · 08-03-2021, 11:01 PM

Sorry it took a little bit to get back to you. I can confirm that switching the preview zoom to 100% fixed my issues.
yes, I know.... I'm always a little late to the party

Thanks Becky! And thanks to Kevin & Diap for looking into this!

Thread Tools	Search this Thread
Show Printable Version Email this Page	Search this Thread: Advanced Search

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Sigil-0.9.10 Released	DiapDealer	Sigil	154	11-19-2018 10:20 PM
Sigil-0.9.7 Released	DiapDealer	Sigil	90	11-10-2016 01:30 PM
Sigil-0.9.6 Released	DiapDealer	Sigil	69	10-29-2016 10:35 AM
Sigil-0.8.900 released for testing - Wait for Sigil-0.8.901	KevinH	Sigil	106	10-04-2015 10:41 AM
Sigil 0.7.0 Released	user_none	Sigil	75	03-03-2013 12:41 PM

08-02-2021, 03:08 PM	#46
KevinH Sigil Developer Posts: 6,313 Karma: 4280902 Join Date: Nov 2009 Device: many	Moving the setZoomFactor to the loadProgress slot allowed me to shrink the needed delay down to 30ms and still consistently see the syncing work. So it helps but we still need the delay. Pushed this to master since it is an improvement.

08-02-2021, 05:30 PM	#47
DiapDealer Grand Sorcerer Posts: 25,173 Karma: 174969608 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	Verified that the fix is working (at 30ms) on my overpowered Linux machine as well as my under-provisioned Windows 10 VM. Let's hope this is sufficient!

08-02-2021, 06:12 PM	#49
DiapDealer Grand Sorcerer Posts: 25,173 Karma: 174969608 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	That's Qt5.15.2 on my Arch Linux machine, by the way. I confirmed the bug still existed before testing the fix. So it's clearly a long-standing problem.

08-03-2021, 11:01 PM	#52
Turtle91 A Hairy Wizard Posts: 2,473 Karma: 14768783 Join Date: Dec 2012 Location: Charleston, SC today Device: iPhone 11/X/6/iPad 1,2 & Air/Surface Pro/Kindle PW	Sorry it took a little bit to get back to you. I can confirm that switching the preview zoom to 100% fixed my issues. yes, I know.... I'm always a little late to the party Thanks Becky! And thanks to Kevin & Diap for looking into this!