Suggestions for Sigil Improvements or New Features

[KevinH]

And interesting enough, line 7 in that Chapter 7 xhtml file is:

<link href="css/style.css" type="text/css" rel="stylesheet"/>

So the bug happens when trying to load the css file that is open but not the first time. The first time it worked. But when using the cache it ties to get the contents of the css/styl.css that is being actively edited and fails!

So my guess is this is due to some Windows specific file locking.

Hmm ...

[KevinH]

@BeckyEbook,

I wonder if this has anything to do with the actual file cache used by WebEngine. Would you please try the following using the debug build.

1. open an epub in Sigil (any epub)
2. Without closing the first Sigil, open a second instance of Sigil and load the Alice epub and run the test but only in the second instance.

Thanks!

[KevinH]

And one more thing to try if you have time ...

In Sigil/src/MainUI/MainWindow.cpp there is a a routine called UpdatePreview.

In this routine I would like you to comment out the line that clears the cache, then try and see if that prevents the error.

From:

void MainWindow::UpdatePreview()
{

    if (m_IsClosing) return;

    m_PreviewTimer.stop();

    DBG qDebug() << "MW: UpdatePreview()";

    QString text;
    QList<ElementIndex> location;
    HTMLResource *html_resource;

    ContentTab *tab = GetCurrentContentTab();
    if (tab != NULL) {

        // Save Tab if update requested from a TextTab tab like CSS or SVG tabs
        if (m_SaveTab) {
            m_SaveTab = false;
            tab->SaveTabContent();
            m_PreviewWindow->ForceFullWebCacheClear();
        }

        html_resource = qobject_cast<HTMLResource *>(tab->GetLoadedResource());

to

void MainWindow::UpdatePreview()
{

    if (m_IsClosing) return;

    m_PreviewTimer.stop();

    DBG qDebug() << "MW: UpdatePreview()";

    QString text;
    QList<ElementIndex> location;
    HTMLResource *html_resource;

    ContentTab *tab = GetCurrentContentTab();
    if (tab != NULL) {

        // Save Tab if update requested from a TextTab tab like CSS or SVG tabs
        if (m_SaveTab) {
            m_SaveTab = false;
            tab->SaveTabContent();
            // m_PreviewWindow->ForceFullWebCacheClear();
        }

        html_resource = qobject_cast<HTMLResource *>(tab->GetLoadedResource());

I am hoping that the problem is related to file locking in the local QtWebEngine file cache.

I have my fiingers crossed, one of these last two will impact the error.

[DiapDealer]

I am mobile-devices-only until tomorrow sometime, or I would test.

[DNSB]

This may not be any help but I tried your suggestion for triggering the issue with my local build using Python 3.14.3/Qt 6.10.2 and the debug log does not show any errors. Up to this point, I hadn't realized that I had updated to 3.14.3 instead of 3.14.2.

[KevinH]

That is strange. The code paths in question do not use python3lib code as far as I can tell, so I have no explanation why one Windows person sees a javascript error that causes the Preview to not properly reload and move to its last location, while another does not see the issue or error at all.

The mystery deepens.

[BeckyEbook]

It was already nighttime where I am, so I only checked it now.

1. The test with two instances still caused an error. (I removed debug for childs form PreviewWindow.cpp, so the log is cleaner).

2. The change in MainWindow.cpp made it work! Note that I am editing the CSS file, but in the preview window I am still in the previous location, which is how it should be.

[DiapDealer]

I'm not where I can build from your fork, but I can also confirm that with the current Sigil, the behavior doesn't change with the second instance of Sigil.

[KevinH]

@BeckyEBook
Thank you.
So it is related to the asynchronous clearing of the cache on Windows somehow.