Use static signal offsets instead of offsets calculated on-the-fly

[pulseview.git] / pv / view / header.cpp

diff --git a/pv/view/header.cpp b/pv/view/header.cpp
index 4eebf1f5dcb25d29224a6885d850ab1f25b4a2e4..125d3c2430ba868d95c0f535e162d9279dda9e90 100644 (file)
--- a/pv/view/header.cpp
+++ b/pv/view/header.cpp
@@ -21,13 +21,16 @@
 #include "header.h"
 #include "view.h"
 
-#include "../signal.h"
+#include "signal.h"
 #include "../sigsession.h"
 
 #include <assert.h>
 
 #include <boost/foreach.hpp>
 
+#include <QColorDialog>
+#include <QInputDialog>
+#include <QMenu>
 #include <QMouseEvent>
 #include <QPainter>
 #include <QRect>
@@ -40,9 +43,16 @@ namespace view {
 
 Header::Header(View &parent) :
        QWidget(&parent),
-       _view(parent)
+       _view(parent),
+       _action_set_name(new QAction(tr("Set &Name..."), this)),
+       _action_set_colour(new QAction(tr("Set &Colour..."), this))
 {
        setMouseTracking(true);
+
+       connect(_action_set_name, SIGNAL(triggered()),
+               this, SLOT(on_action_set_name_triggered()));
+       connect(_action_set_colour, SIGNAL(triggered()),
+               this, SLOT(on_action_set_colour_triggered()));
 }
 
 void Header::paintEvent(QPaintEvent *event)
@@ -54,19 +64,17 @@ void Header::paintEvent(QPaintEvent *event)
        QPainter painter(this);
        painter.setRenderHint(QPainter::Antialiasing);
 
-       int offset = -_view.v_offset();
+       const int v_offset = _view.v_offset();
        BOOST_FOREACH(const shared_ptr<Signal> s, sigs)
        {
                assert(s);
 
                const QRect signal_heading_rect(
-                       0, offset, w, View::SignalHeight);
+                       0, s->get_v_offset() - v_offset,
+                       w, View::SignalHeight);
 
                s->paint_label(painter, signal_heading_rect,
-                       s->pt_in_label_rect(painter,
-                               signal_heading_rect, _mouse_point));
-
-               offset += View::SignalHeight;
+                       s->pt_in_label_rect(signal_heading_rect, _mouse_point));
        }
 
        painter.end();
@@ -85,5 +93,60 @@ void Header::leaveEvent(QEvent *event)
        update();
 }
 
+void Header::contextMenuEvent(QContextMenuEvent *event)
+{
+       const int w = width();
+       const vector< shared_ptr<Signal> > &sigs =
+               _view.session().get_signals();
+
+       const int v_offset = _view.v_offset();
+       BOOST_FOREACH(const shared_ptr<Signal> s, sigs)
+       {
+               assert(s);
+
+               const QRect signal_heading_rect(
+                       0, s->get_v_offset() - v_offset,
+                       w, View::SignalHeight);
+
+               if(s->pt_in_label_rect(signal_heading_rect, _mouse_point)) {
+                       QMenu menu(this);
+                       menu.addAction(_action_set_name);
+                       menu.addAction(_action_set_colour);
+
+                       _context_signal = s;
+                       menu.exec(event->globalPos());
+                       _context_signal.reset();
+
+                       break;
+               }
+       }
+}
+
+void Header::on_action_set_name_triggered()
+{
+       shared_ptr<view::Signal> context_signal = _context_signal;
+       if(!context_signal)
+               return;
+
+       const QString new_label = QInputDialog::getText(this, tr("Set Name"),
+               tr("Name"), QLineEdit::Normal, context_signal->get_name());
+
+       if(!new_label.isEmpty())
+               context_signal->set_name(new_label);
+}
+
+void Header::on_action_set_colour_triggered()
+{
+       shared_ptr<view::Signal> context_signal = _context_signal;
+       if(!context_signal)
+               return;
+
+       const QColor new_colour = QColorDialog::getColor(
+               context_signal->get_colour(), this, tr("Set Colour"));
+
+       if(new_colour.isValid())
+               context_signal->set_colour(new_colour);
+}
+
 } // namespace view
 } // namespace pv