View Single Post
Old 08-30-2024, 11:07 AM   #7
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,874
Karma: 6120478
Join Date: Nov 2009
Device: many
Yes we are missing a return here in the Paste routine in MainWindow.cpp:

Code:
if (!html.isEmpty()) {
        QMessageBox msgBox(QMessageBox::Question,
          tr("Clipboard contains HTML formatting"),
          tr("Do you want to paste clipboard data as plain text?"),
          QMessageBox::Yes|QMessageBox::No|QMessageBox::Cancel);
        msgBox.setDefaultButton(QMessageBox::Yes);
        // populate the detailed text window - by HTML
        msgBox.setDetailedText(html);
        int rv = msgBox.exec();
        if (rv == QMessageBox::Yes) {
            m_WebView->triggerPageAction(QWebEnginePage::PasteAndMatchStyle);
        } else if (rv == QMessageBox::No) {
            m_WebView->triggerPageAction(QWebEnginePage::Paste);
        } else {
            // they have hit Cancel so paste nothing
            return;
        }
        return;   // fix missing return
    }
I will push that fix to PageEdit master.
KevinH is online now   Reply With Quote