diff --git a/stats/statshelper.h b/stats/statshelper.h index 29dc083a4..f5d7cbcea 100644 --- a/stats/statshelper.h +++ b/stats/statshelper.h @@ -1,29 +1,12 @@ // SPDX-License-Identifier: GPL-2.0 -// Helper functions to render the stats. Currently only -// contains a small template to create scene-items. This -// is for historical reasons to ease transition from QtCharts -// and might be removed. +// Helper functions to render the stats. Currently contains +// QSGNode template jugglery to overcome API flaws. #ifndef STATSHELPER_H #define STATSHELPER_H #include -#include #include -template -T *createItem(QGraphicsScene *scene, Args&&... args) -{ - T *res = new T(std::forward(args)...); - scene->addItem(res); - return res; -} - -template -std::unique_ptr createItemPtr(QGraphicsScene *scene, Args&&... args) -{ - return std::unique_ptr(createItem(scene, std::forward(args)...)); -} - // In general, we want chart items to be hideable. For example to show/hide // labels on demand. Very sadly, the QSG API is absolutely terrible with // respect to temporarily disabling. Instead of simply having a flag, diff --git a/stats/statsview.cpp b/stats/statsview.cpp index efb530e45..550219286 100644 --- a/stats/statsview.cpp +++ b/stats/statsview.cpp @@ -20,8 +20,6 @@ #include "core/subsurface-qt/divelistnotifier.h" #include -#include -#include #include #include #include @@ -88,10 +86,9 @@ class RootNode : public QSGNode { public: RootNode(QQuickWindow *w); - QSGRectangleNode *backgroundNode; // solid background - QSGImageNode *imageNode; // imageNode to plot QGRaphicsScene on. Remove in due course. + std::unique_ptr backgroundNode; // solid background // We entertain one node per Z-level. - std::array zNodes; + std::array, (size_t)ChartZValue::Count> zNodes; }; RootNode::RootNode(QQuickWindow *w) @@ -99,16 +96,14 @@ RootNode::RootNode(QQuickWindow *w) // Add a background rectangle with a solid color. This could // also be done on the widget level, but would have to be done // separately for desktop and mobile, so do it here. - backgroundNode = w->createRectangleNode(); + backgroundNode.reset(w->createRectangleNode()); backgroundNode->setColor(backgroundColor); - appendChildNode(backgroundNode); + appendChildNode(backgroundNode.get()); - for (QSGNode *&zNode: zNodes) { - zNode = new QSGNode; - appendChildNode(zNode); + for (auto &zNode: zNodes) { + zNode.reset(new QSGNode); + appendChildNode(zNode.get()); } - imageNode = w->createImageNode(); - zNodes[(int)ChartZValue::Series]->appendChildNode(imageNode); } QSGNode *StatsView::updatePaintNode(QSGNode *oldNode, QQuickItem::UpdatePaintNodeData *) @@ -133,11 +128,6 @@ QSGNode *StatsView::updatePaintNode(QSGNode *oldNode, QQuickItem::UpdatePaintNod item->dirtyPrev = nullptr; } - img->fill(Qt::transparent); - scene.render(painter.get()); - texture.reset(window()->createTextureFromImage(*img, QQuickWindow::TextureHasAlphaChannel)); - n->imageNode->setTexture(texture.get()); - n->imageNode->setRect(rect); return n; } @@ -194,16 +184,6 @@ QRectF StatsView::plotArea() const void StatsView::plotAreaChanged(const QSizeF &s) { - // Make sure that image is at least one pixel wide / high, otherwise - // the painter starts acting up. - int w = std::max(1, static_cast(floor(s.width()))); - int h = std::max(1, static_cast(floor(s.height()))); - scene.setSceneRect(QRectF(0, 0, static_cast(w), static_cast(h))); - painter.reset(); - img.reset(new QImage(w, h, QImage::Format_ARGB32)); - painter.reset(new QPainter(img.get())); - painter->setRenderHint(QPainter::Antialiasing); - double left = sceneBorder; double top = sceneBorder; double right = s.width() - sceneBorder; @@ -363,7 +343,7 @@ void StatsView::plot(const StatsState &stateIn) { state = stateIn; plotChart(); - plotAreaChanged(scene.sceneRect().size()); + plotAreaChanged(boundingRect().size()); update(); } diff --git a/stats/statsview.h b/stats/statsview.h index 1736af219..bf8ad48a2 100644 --- a/stats/statsview.h +++ b/stats/statsview.h @@ -5,7 +5,6 @@ #include "statsstate.h" #include #include -#include #include #include #include @@ -58,10 +57,6 @@ private: // QtQuick related things QRectF plotRect; QSGNode *updatePaintNode(QSGNode *oldNode, QQuickItem::UpdatePaintNodeData *updatePaintNodeData) override; - std::unique_ptr img; - std::unique_ptr painter; - QGraphicsScene scene; - std::unique_ptr texture; void plotAreaChanged(const QSizeF &size); void reset(); // clears all series and axes