From 5eefad6ab66a01939803a02f082365066f63395d Mon Sep 17 00:00:00 2001
From: Victor Zarubkin <v.s.zarubkin@gmail.com>
Date: Wed, 14 Mar 2018 00:31:07 +0300
Subject: [PATCH] #31 [UI] Highlight current selected block boundaries on
 arbitrary value chart

---
 profiler_gui/arbitrary_value_inspector.cpp | 72 ++++++++++++++++++----
 profiler_gui/globals_qobjects.h            |  1 +
 profiler_gui/main_window.cpp               |  2 +
 3 files changed, 64 insertions(+), 11 deletions(-)

diff --git a/profiler_gui/arbitrary_value_inspector.cpp b/profiler_gui/arbitrary_value_inspector.cpp
index 2b1dc76..2ec755b 100644
--- a/profiler_gui/arbitrary_value_inspector.cpp
+++ b/profiler_gui/arbitrary_value_inspector.cpp
@@ -646,22 +646,72 @@ void ArbitraryValuesChartItem::paint(QPainter* _painter, const QStyleOptionGraph
     _painter->save();
     _painter->setTransform(QTransform::fromScale(1.0 / currentScale, 1), true);
 
-    if (!bindMode)
-        paintImage(_painter);
-    else
-        paintImage(_painter, currentScale, widget->minimum(), widget->maximum(), widget->value(), widget->sliderWidth());
-
     const auto font_h = widget->fontHeight();
-    QRectF rect(0, m_boundingRect.top() - widget->margin(), width - 3, m_boundingRect.height() + widget->margins());
-    _painter->setPen(profiler_gui::TEXT_COLOR);
-    _painter->drawText(rect, Qt::AlignLeft | Qt::AlignTop, bindMode ? " Mode: Zoom" : " Mode: Overview");
+
+    const auto drawImage = [=] {
+        if (!bindMode)
+            paintImage(_painter);
+        else
+            paintImage(_painter, currentScale, widget->minimum(), widget->maximum(), widget->value(), widget->sliderWidth());
+    };
 
     if (!EASY_GLOBALS.scene.empty)
     {
+        if (!profiler_gui::is_max(EASY_GLOBALS.selected_block) && m_chartType == ChartType::Regular)
+        {
+            const auto& block = easyBlocksTree(EASY_GLOBALS.selected_block);
+            qreal left = PROF_MICROSECONDS(qreal(block.node->begin() - EASY_GLOBALS.begin_time)) * currentScale;
+            qreal right = PROF_MICROSECONDS(qreal(block.node->end() - EASY_GLOBALS.begin_time)) * currentScale;
+
+            if (bindMode)
+            {
+                const auto scale = widget->range() / widget->sliderWidth();
+                const auto offset = widget->value() * currentScale;
+
+                left -= offset;
+                right -= offset;
+                left *= scale;
+                right *= scale;
+            }
+
+            const auto leftBorder = m_boundingRect.left() * currentScale;
+            const auto rightBorder = leftBorder + width - 2;
+            const auto l = std::max(left, leftBorder);
+            const auto r = std::min(right, rightBorder);
+
+            const auto sceneRect = scene()->sceneRect();
+            const auto w = r - l;
+            if (w > 1)
+            {
+                const auto color = (easyDescriptor(block.node->id()).color() & 0x00ffffff) | 0x30000000;
+                _painter->fillRect(QRectF(l, sceneRect.top() - 1, w, sceneRect.height() + 2), QColor::fromRgba(color));
+            }
+
+            drawImage();
+
+            _painter->setPen(QPen(QColor::fromRgba(0xb0000000), 2, Qt::DotLine));
+            _painter->drawLine(QLineF(l, sceneRect.top(), l, sceneRect.bottom()));
+
+            if (w > 1)
+                _painter->drawLine(QLineF(r, sceneRect.top(), r, sceneRect.bottom()));
+        }
+        else
+        {
+            drawImage();
+        }
+
         const auto range = bindMode ? widget->sliderWidth() : widget->range();
         paintMouseIndicator(_painter, m_boundingRect.top(), bottom, width,
                             m_boundingRect.height(), font_h, widget->value(), range);
     }
+    else
+    {
+        drawImage();
+    }
+
+    QRectF rect(0, m_boundingRect.top() - widget->margin(), width - 3, m_boundingRect.height() + widget->margins());
+    _painter->setPen(profiler_gui::TEXT_COLOR);
+    _painter->drawText(rect, Qt::AlignLeft | Qt::AlignTop, bindMode ? " Mode: Zoom" : " Mode: Overview");
 
     _painter->setPen(Qt::darkGray);
     _painter->drawLine(QLineF(0, bottom, width, bottom));
@@ -2088,15 +2138,14 @@ void ArbitraryValuesWidget::onSelectedThreadChanged(profiler::thread_id_t)
 
 void ArbitraryValuesWidget::onSelectedBlockChanged(uint32_t)
 {
+    m_chart->scene()->update();
+
     if (profiler_gui::is_max(EASY_GLOBALS.selected_block))
     {
         onSelectedBlockIdChanged(EASY_GLOBALS.selected_block_id);
         return;
     }
 
-    if (!profiler_gui::is_max(EASY_GLOBALS.selected_block))
-        return;
-
     // TODO: find item corresponding to selected_block and make it bold
 }
 
@@ -2536,6 +2585,7 @@ void ArbitraryValuesWidget::onOpenInNewWindowClicked(bool)
     dialog->setAttribute(Qt::WA_DeleteOnClose, true);
     dialog->setWindowTitle("EasyProfiler");
     connect(&EASY_GLOBALS.events, &profiler_gui::GlobalSignals::allDataGoingToBeDeleted, dialog, &QDialog::reject);
+    connect(&EASY_GLOBALS.events, &profiler_gui::GlobalSignals::closeEvent, dialog, &QDialog::reject);
 
     auto viewer = new ArbitraryValuesWidget(m_checkedItems, m_treeWidget->currentItem(), m_threadId, m_blockIndex, m_blockId, dialog);
 
diff --git a/profiler_gui/globals_qobjects.h b/profiler_gui/globals_qobjects.h
index 4749227..89dfb70 100644
--- a/profiler_gui/globals_qobjects.h
+++ b/profiler_gui/globals_qobjects.h
@@ -73,6 +73,7 @@ namespace profiler_gui {
 
     signals:
 
+        void closeEvent();
         void allDataGoingToBeDeleted();
         void fileOpened();
 
diff --git a/profiler_gui/main_window.cpp b/profiler_gui/main_window.cpp
index fdae4a2..20f5fe8 100644
--- a/profiler_gui/main_window.cpp
+++ b/profiler_gui/main_window.cpp
@@ -1332,6 +1332,8 @@ void MainWindow::closeEvent(QCloseEvent* close_event)
         }
     }
 
+    emit EASY_GLOBALS.events.closeEvent();
+
     if (m_descTreeDialog.ptr != nullptr)
     {
         m_descTreeDialog.ptr->reject();