X-Git-Url: https://git.quassel-irc.org/?p=quassel.git;a=blobdiff_plain;f=src%2Fqtui%2Fwebpreviewitem.cpp;h=e987e3de443b10d018903e27de34098fa3a72e09;hp=1d7a27cd619a136d19857e94d86897945474705d;hb=1f21c1f9613031ae263eeed0c4883bfcd5488343;hpb=921e54680da16fcf2adb7a90506875aceb6633a4

diff --git a/src/qtui/webpreviewitem.cpp b/src/qtui/webpreviewitem.cpp
index 1d7a27cd..e987e3de 100644
--- a/src/qtui/webpreviewitem.cpp
+++ b/src/qtui/webpreviewitem.cpp
@@ -1,5 +1,5 @@
 /***************************************************************************
- *   Copyright (C) 2005-2015 by the Quassel Project                        *
+ *   Copyright (C) 2005-2019 by the Quassel Project                        *
  *   devel@quassel-irc.org                                                 *
  *                                                                         *
  *   This program is free software; you can redistribute it and/or modify  *
@@ -20,25 +20,59 @@
 
 #include "webpreviewitem.h"
 
-#ifdef HAVE_WEBKIT
+#ifdef HAVE_WEBENGINE
+#    include <QWebEngineSettings>
+#    include <QWebEngineView>
+#elif defined HAVE_WEBKIT
+#    include <QWebSettings>
+#    include <QWebView>
+#endif
 
-#include <QGraphicsProxyWidget>
-#include <QPainter>
-#include <QWebView>
-#include <QWebSettings>
+#if defined HAVE_WEBKIT || defined HAVE_WEBENGINE
 
-WebPreviewItem::WebPreviewItem(const QUrl &url)
-    : QGraphicsItem(0), // needs to be a top level item as we otherwise cannot guarantee that it's on top of other chatlines
+#    include <QGraphicsProxyWidget>
+#    include <QPainter>
+
+#    ifdef HAVE_WEBENGINE
+// QGraphicsProxyWidget does not stay synced with QWebEngineView, therefore we modify QWebEngineView
+// and manually trigger an update of the proxyItem on UpdateRequest Event. See: http://stackoverflow.com/a/30920209
+// and http://lists.qt-project.org/pipermail/development/2016-March/025280.html (At least in Qt5.6)
+class CustomWebView : public QWebEngineView
+{
+private:
+    QGraphicsProxyWidget* proxyItem;
+
+public:
+    CustomWebView(QGraphicsProxyWidget* pItem) { proxyItem = pItem; }
+    bool event(QEvent* event) override
+    {
+        if (event->type() == QEvent::UpdateRequest) {
+            proxyItem->update();
+        }
+
+        return QWebEngineView::event(event);
+    }
+};
+#    endif
+
+WebPreviewItem::WebPreviewItem(const QUrl& url)
+    : QGraphicsItem(nullptr)
+    ,  // needs to be a top level item as we otherwise cannot guarantee that it's on top of other chatlines
     _boundingRect(0, 0, 400, 300)
 {
     qreal frameWidth = 5;
 
-    QWebView *webView = new QWebView;
+    QGraphicsProxyWidget* proxyItem = new QGraphicsProxyWidget(this);
+#    ifdef HAVE_WEBENGINE
+    QWebEngineView* webView = new CustomWebView(proxyItem);
+    webView->settings()->setAttribute(QWebEngineSettings::JavascriptEnabled, false);
+#    elif defined HAVE_WEBKIT
+    QWebView* webView = new QWebView;
     webView->settings()->setAttribute(QWebSettings::JavascriptEnabled, false);
+#    endif
     webView->load(url);
     webView->setDisabled(true);
     webView->resize(1000, 750);
-    QGraphicsProxyWidget *proxyItem = new QGraphicsProxyWidget(this);
     proxyItem->setWidget(webView);
     proxyItem->setAcceptHoverEvents(false);
 
@@ -50,10 +84,10 @@ WebPreviewItem::WebPreviewItem(const QUrl &url)
     setZValue(30);
 }
 
-
-void WebPreviewItem::paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget)
+void WebPreviewItem::paint(QPainter* painter, const QStyleOptionGraphicsItem* option, QWidget* widget)
 {
-    Q_UNUSED(option); Q_UNUSED(widget);
+    Q_UNUSED(option);
+    Q_UNUSED(widget);
     painter->setClipRect(boundingRect());
     painter->setPen(QPen(Qt::black, 5));
     painter->setBrush(Qt::black);
@@ -61,5 +95,4 @@ void WebPreviewItem::paint(QPainter *painter, const QStyleOptionGraphicsItem *op
     painter->drawRoundedRect(boundingRect(), 10, 10);
 }
 
-
-#endif //#ifdef HAVE_WEBKIT
+#endif  //#ifdef HAVE_WEBKIT || HAVE_WEBENGINE