From 0bd3ee414d434c8f0973659420e0117a6cc59425 Mon Sep 17 00:00:00 2001
From: Lieven Hey <lieven.hey@kdab.com>
Date: Fri, 15 Nov 2024 15:09:51 +0100
Subject: [PATCH] fix: broken flamegraph

Apparently when running X11 the flamgraph doesn't clear the background
correctly. This patch resets the background brush which clears the
background correctly.

fixes: #624
---
 src/flamegraph.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/flamegraph.cpp b/src/flamegraph.cpp
index c3e5f4b3..3ba1d777 100644
--- a/src/flamegraph.cpp
+++ b/src/flamegraph.cpp
@@ -1189,11 +1189,11 @@ void FlameGraph::selectItem(FrameGraphicsItem* item)
     // then layout all items below the selected on
     layoutItems(item);
 
-#if QT_VERSION <= QT_VERSION_CHECK(5, 15, 12) && QT_VERSION > QT_VERSION_CHECK(5, 15, 10)
+    // #if QT_VERSION <= QT_VERSION_CHECK(5, 15, 12) && QT_VERSION > QT_VERSION_CHECK(5, 15, 10)
     // the scene background doesn't get cleared correctly when using qt 5.15.12
     // this doesn't happen in 5.15.10 and 5.15.13
     m_scene->setBackgroundBrush(QBrush());
-#endif
+    // #endif
 
     // Triggers a refresh of the scene's bounding rect without going via the
     // event loop. This makes the centerOn call below work as expected in all cases.