Suggestions for Sigil Improvements or New Features

[KevinH]

Your debug log shows the location being set properly and used once but then somehow the location (tag name and child number info) is lost. And exactly as you said, there are no javascript console errors.

Very strange. I will have to try and run a Windows build done by CI tomorrow on my old Linux box to see if I can see where the location is being lost.

[DNSB]

Recompiled and still get the preview moving to the top of the file when I open the stylesheet. However, I did notice that if I click on the CodeView tab for the HTML file and then click in the code, preview moves back to that position.

[theducks]

I did not run the RED test.
But I did run my normal Adjust the CSS sequence with the preview about mid point (where I want the adjustment). The adjustments took, I saw the preview refresh the screen AND STAY where I left it.
Previously, it would revert to the top.
W10 x64

BTW it was EPUB2 if that matters

[KevinH]

Arrgghhh! Thanks for checking. We are back to square one. For some reason the location is getting stepped on.

I will try adding some key debug in places and trying a Windows CI build in a vm on my old Linux box.

Something unexpected is going on. Not sure what.

[KevinH]

Originally Posted by theducks

I did not run the RED test.
But I did run my normal Adjust the CSS sequence with the preview about mid point (where I want the adjustment). The adjustments took, I saw the preview refresh the screen AND STAY where I left it.
Previously, it would revert to the top.
W10 x64

BTW it was EPUB2 if that matters

Great thanks for testing and reporting back. You are seeing exactly what I hoped for but DNSB is showing it doesn't work. Epub2 vs Epub3 should not matter here.

So it sounds like a race between 2 events, when one wins, everything works but when the other wins, it fails.

I thought this was going to be an easy fix but it is going to take some digging.

[DiapDealer]

Remember you can manually run just the Windows CI action and it will build an installer you can use. I should also be around this weekend to help do some Windows debugging.

[DNSB]

Before I installed the new compile of Sigil, I ran the previous one and noticed that I got two error messages at the end.

Debug: Caching:  "OEBPS/Chapter-07.xhtml"
Debug: Caching:  "OEBPS/Chapter-07.xhtml"
Critical: Request Autofill.enable failed. {"code":-32601,"message":"'Autofill.enable' wasn't found"}
Critical: Request Autofill.setAddresses failed. {"code":-32601,"message":"'Autofill.setAddresses' wasn't found"}

[DNSB]

Installed a compile of Sigil from a download from when the debug was first enabled. No issues with moving around in preview when shifting to the stylesheet though preview did refresh each time. I added color: blue to the p tag and it applied immediately.

OTOH, I put the same executable on an old laptop. The location in preview bobbed around.

I suspect a race conditions and those tend to be so much fun...

[KevinH]

Originally Posted by DiapDealer

Remember you can manually run just the Windows CI action and it will build an installer you can use. I should also be around this weekend to help do some Windows debugging.

Great! As I do think it is a event race for some unknown reason.

Last night, I pushed to Sigil master some selected extra debug info.

And yes I should have done the CI deploy manually on just Windows and forgot I could do that so I did a full deploy.

If you get a chance using the very last deploy build of Sigil on Windows, I would love to see the debug log from running the Alice epub test described earlier, I have the same output from my Mac (and Linux) to compare against.

Thanks!