Suggestions for Sigil Improvements or New Features

[KevinH]

FWIW, I tested the DEBUG7 build on my old iMac on Linux using a Windows 11 VM and it passed all my tests!

[BeckyEbook]

I confirm that Debug7 works. I will compile my own version and test it during normal EPUB file editing.

[KevinH]

Originally Posted by BeckyEbook

I confirm that Debug7 works. I will compile my own version and test it during normal EPUB file editing.

Great! Thank you!

If you grab current master now, you should get a version with updated profile caches and all debug turned off.

Do not forget to get rid of your SIGIL_DEBUG_LOGFILE env var.

I have tested both test cases we have used here as well as pdf display and use of local storage.

With current master, so far so good!

[DNSB]

I compile the debug7 code and pretty much everything looked okay. The first time I opened the blank document, in the log file, there was an error message:

Error: URLInterceptorDebug: In ViewPreview ConnectSignalsToSlots

However this error never re-occurred. I ran the tests on my physical box and in two VMs running Windows 11.

[KevinH]

Your URL document error is interesting:

Debug: Error: URLInterceptor cannot determine book folder so all file requests blocked

It is generated by our Sigil/src/Misc/URLInterceptor.cpp code whose job it is to prevent anything in the epub from loading anything NOT in the current epub folder, mathjax folder, or Sigil preferences folder.

It is warning that it can not find a disc based book folder for the empty epub that Sigil opens at startup. Since it can't find the folder, it can not tell if any link is to something outside of it and to be safe denies that request.

This may just be an artifact of an unused empty epub. Either way I will take a look see.

Thanks for testing and reporting back.

[KevinH]

@DNSB,
If you think you can recreate this url error with another empty book start up of Sigil.

Please edit Sigil/src/Misc/URLInterceptor.cpp and in line 33, change this line:

#define DBG if(0)

in that file to be

#define DBG if(1)

And then in line 111, remove the beginning // from this line:

// DBG qDebug() << "path_to_book: " << path_to_book;

to become:

DBG qDebug() << "path_to_book: " << path_to_book;


and then recompile and run the same tests with a fresh debug log file and post it here.

That way we might be able to see what url request is triggering your error, assuming it is repeatable.

Thanks!

[DNSB]

I will do that change and recompile.