X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fview%2Fheader.cpp;h=dec5c20cbabfad042ec785886a2a2be9ca747e40;hp=b85b1bb1c0a03cc5ed11249d0e4dd96df2865d26;hb=9d28da5ae394e572dedf0ec22c17fd259825e1c9;hpb=01fd32630269c1cfb99eb2b0d6060d7d19a42475

diff --git a/pv/view/header.cpp b/pv/view/header.cpp
index b85b1bb1..dec5c20c 100644
--- a/pv/view/header.cpp
+++ b/pv/view/header.cpp
@@ -29,13 +29,13 @@
 #include <boost/foreach.hpp>
 
 #include <QApplication>
-#include <QColorDialog>
-#include <QInputDialog>
 #include <QMenu>
 #include <QMouseEvent>
 #include <QPainter>
 #include <QRect>
 
+#include <pv/widgets/popup.h>
+
 using namespace boost;
 using namespace std;
 
@@ -44,16 +44,11 @@ namespace view {
 
 Header::Header(View &parent) :
 	MarginWidget(parent),
-	_action_set_name(new QAction(tr("Set &Name..."), this)),
-	_action_set_colour(new QAction(tr("Set &Colour..."), this))
+	_dragging(false)
 {
+	setFocusPolicy(Qt::ClickFocus);
 	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()));
-
 	connect(&_view.session(), SIGNAL(signals_changed()),
 		this, SLOT(on_signals_changed()));
 
@@ -159,10 +154,28 @@ void Header::mousePressEvent(QMouseEvent *event)
 
 void Header::mouseReleaseEvent(QMouseEvent *event)
 {
+	using pv::widgets::Popup;
+
 	assert(event);
 	if (event->button() == Qt::LeftButton) {
+		if (_dragging)
+			_view.normalize_layout();
+		else
+		{
+			const shared_ptr<Trace> mouse_over_trace =
+				get_mouse_over_trace(event->pos());
+			if (mouse_over_trace) {
+				Popup *const p =
+					mouse_over_trace->create_popup(&_view);
+				p->set_position(mapToGlobal(QPoint(width(),
+					mouse_over_trace->get_y())),
+					Popup::Right);
+				p->show();
+			}
+		}
+
+		_dragging = false;
 		_drag_traces.clear();
-		_view.normalize_layout();
 	}
 }
 
@@ -171,8 +184,18 @@ void Header::mouseMoveEvent(QMouseEvent *event)
 	assert(event);
 	_mouse_point = event->pos();
 
+	if (!(event->buttons() & Qt::LeftButton))
+		return;
+
+	if ((event->pos() - _mouse_down_point).manhattanLength() <
+		QApplication::startDragDistance())
+		return;
+
 	// Move the signals if we are dragging
-	if (!_drag_traces.empty()) {
+	if (!_drag_traces.empty())
+	{
+		_dragging = true;
+
 		const int delta = event->pos().y() - _mouse_down_point.y();
 
 		for (std::list<std::pair<boost::weak_ptr<Trace>,
@@ -209,44 +232,25 @@ void Header::contextMenuEvent(QContextMenuEvent *event)
 {
 	const shared_ptr<Trace> t = get_mouse_over_trace(_mouse_point);
 
-	if (!t)
-		return;
-
-	QMenu menu(this);
-	menu.addAction(_action_set_name);
-	menu.addAction(_action_set_colour);
-
-	_context_trace = t;
-	menu.exec(event->globalPos());
-	_context_trace.reset();
+	if (t)
+		t->create_context_menu(this)->exec(event->globalPos());
 }
 
-void Header::on_action_set_name_triggered()
+void Header::keyPressEvent(QKeyEvent *e)
 {
-	bool ok = false;
+	assert(e);
 
-	shared_ptr<view::Trace> context_trace = _context_trace;
-	if (!context_trace)
-		return;
-
-	const QString new_label = QInputDialog::getText(this, tr("Set Name"),
-		tr("Name"), QLineEdit::Normal, context_trace->get_name(), &ok);
-
-	if (ok)
-		context_trace->set_name(new_label);
-}
-
-void Header::on_action_set_colour_triggered()
-{
-	shared_ptr<view::Trace> context_trace = _context_trace;
-	if (!context_trace)
-		return;
-
-	const QColor new_colour = QColorDialog::getColor(
-		context_trace->get_colour(), this, tr("Set Colour"));
-
-	if (new_colour.isValid())
-		context_trace->set_colour(new_colour);
+	switch (e->key())
+	{
+	case Qt::Key_Delete:
+	{
+		const vector< shared_ptr<Trace> > traces(_view.get_traces());
+		BOOST_FOREACH(const shared_ptr<Trace> t, traces)
+			if (t->selected())
+				t->delete_pressed();	
+		break;
+	}
+	}
 }
 
 void Header::on_signals_changed()
@@ -254,7 +258,12 @@ void Header::on_signals_changed()
 	const vector< shared_ptr<Trace> > traces(_view.get_traces());
 	BOOST_FOREACH(shared_ptr<Trace> t, traces) {
 		assert(t);
-		connect(t.get(), SIGNAL(text_changed()), this, SLOT(update()));
+		connect(t.get(), SIGNAL(visibility_changed()),
+			this, SLOT(update()));
+		connect(t.get(), SIGNAL(text_changed()),
+			this, SLOT(update()));
+		connect(t.get(), SIGNAL(colour_changed()),
+			this, SLOT(update()));
 	}
 }