D698: Port from WebKit to WebEngine.

Anthony Fieroni noreply at phabricator.kde.org
Fri Dec 29 06:13:10 UTC 2017


anthonyfieroni added inline comments.

INLINE COMMENTS

> CMakeLists.txt:29-30
> +
>        ${Qt5WebKitWidgets_LIBRARIES}
>        ${Qt5WebKit_LIBRARIES}
>        ${Qt5Sql_LIBRARIES}

You now depends on 2 engines, WebEngine and Webkit. So to be port complete remove Webkit or make it optional (last one will introduce many compile time conditional code) Situation is the same for other executables.

> MarbleLegendBrowser.cpp:86-88
> +//TODO: no easy way to delegate links
> +//    page()->setLinkDelegationPolicy( QWebEnginePage::DelegateAllLinks );
> +//    connect( this, SIGNAL(linkClicked(QUrl)), this, SLOT(openLinkExternally(QUrl)) );

Docs <https://doc.qt.io/qt-5/qtwebenginewidgets-qtwebkitportingguide.html> says it will be overridden  QWebEnginePage::acceptNavigationRequest

> MarbleLegendBrowser.cpp:398
> +//TODO: No api to support extracting DOM elements.
> +//    QWebElement box = page()->mainFrame()->findFirstElement("input[name="+name+']');
> +//    if (!box.isNull()) {

http://doc.qt.io/archives/qt-5.5/qwebframe.html#findFirstElement ?

> PopupItem.cpp:400
> +
> +//TODO: No api for printing the web page.
> +//    QPrinter printer;

It has https://doc.qt.io/qt-5/qwebenginepage.html#print

REPOSITORY
  R34 Marble

REVISION DETAIL
  https://phabricator.kde.org/D698

To: cmihalache, #marble
Cc: anthonyfieroni, mnafees, #kde_edu, kossebau, rahn, nienhueser, #marble, jalvarez, tjakobi, shentey, chaz6, dkolozsvari, narvaez, cmihalache, apol, marble-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20171229/4854a8f9/attachment.html>


More information about the Marble-devel mailing list