X-Git-Url: http://git.code-monkey.de/?a=blobdiff_plain;f=pv%2Fview%2Fheader.cpp;h=2ab8ed986568ba4bfeaa36043840b723541ce867;hb=dc747705ac06c9e9ab582b6b8b7cfc31003128e0;hp=ffe05a1c674335f92479e85997d69ec0347cc46b;hpb=5c47d6c11c336a963c0251b55313bab3fababcaa;p=pulseview.git

diff --git a/pv/view/header.cpp b/pv/view/header.cpp
index ffe05a1..2ab8ed9 100644
--- a/pv/view/header.cpp
+++ b/pv/view/header.cpp
@@ -123,12 +123,11 @@ void Header::paintEvent(QPaintEvent*)
 	QPainter painter(this);
 	painter.setRenderHint(QPainter::Antialiasing);
 
-	const bool dragging = !_drag_row_items.empty();
 	for (const shared_ptr<RowItem> r : row_items)
 	{
 		assert(r);
 
-		const bool highlight = !dragging &&
+		const bool highlight = !_dragging &&
 			r->label_rect(w).contains(_mouse_point);
 		r->paint_label(painter, w, highlight);
 	}
@@ -136,70 +135,85 @@ void Header::paintEvent(QPaintEvent*)
 	painter.end();
 }
 
+void Header::mouseLeftPressEvent(QMouseEvent *event)
+{
+	(void)event;
+
+	const bool ctrl_pressed =
+		QApplication::keyboardModifiers() & Qt::ControlModifier;
+
+	// Clear selection if control is not pressed and this item is unselected
+	if ((!_mouse_down_item || !_mouse_down_item->selected()) &&
+		!ctrl_pressed)
+		for (shared_ptr<RowItem> r : _view)
+			r->select(false);
+
+	// Set the signal selection state if the item has been clicked
+	if (_mouse_down_item) {
+		if (ctrl_pressed)
+			_mouse_down_item->select(!_mouse_down_item->selected());
+		else
+			_mouse_down_item->select(true);
+	}
+
+	// Save the offsets of any signals which will be dragged
+	for (const shared_ptr<RowItem> r : _view)
+		if (r->selected())
+			r->drag();
+
+	selection_changed();
+	update();
+}
+
 void Header::mousePressEvent(QMouseEvent *event)
 {
 	assert(event);
 
-	if (event->button() & Qt::LeftButton) {
-		_mouse_down_point = event->pos();
+	_mouse_down_point = event->pos();
+	_mouse_down_item = get_mouse_over_row_item(event->pos());
 
-		// Save the offsets of any signals which will be dragged
-		for (const shared_ptr<RowItem> r : _view)
-			if (r->selected())
-				_drag_row_items.push_back(
-					make_pair(r, r->v_offset()));
-	}
+	if (event->button() & Qt::LeftButton)
+		mouseLeftPressEvent(event);
+}
+
+void Header::mouseLeftReleaseEvent(QMouseEvent *event)
+{
+	assert(event);
 
-	// Select the signal if it has been clicked
-	const shared_ptr<RowItem> mouse_over_row_item =
+	const bool ctrl_pressed =
+		QApplication::keyboardModifiers() & Qt::ControlModifier;
+
+	// Unselect everything if control is not pressed
+	const shared_ptr<RowItem> mouse_over =
 		get_mouse_over_row_item(event->pos());
-	if (mouse_over_row_item) {
-		if (mouse_over_row_item->selected())
-			mouse_over_row_item->select(false);
-		else {
-			mouse_over_row_item->select(true);
-
-			if (~QApplication::keyboardModifiers() &
-				Qt::ControlModifier)
-				_drag_row_items.clear();
-
-			// Add the signal to the drag list
-			if (event->button() & Qt::LeftButton)
-				_drag_row_items.push_back(
-					make_pair(mouse_over_row_item,
-					mouse_over_row_item->v_offset()));
-		}
-	}
 
-	if (~QApplication::keyboardModifiers() & Qt::ControlModifier) {
-		// Unselect all other signals because the Ctrl is not
-		// pressed
-		for (const shared_ptr<RowItem> r : _view)
-			if (r != mouse_over_row_item)
-				r->select(false);
+	for (auto &r : _view)
+		r->drag_release();
+
+	if (_dragging)
+		_view.normalize_layout();
+	else
+	{
+		if (!ctrl_pressed) {
+			for (shared_ptr<RowItem> r : _view)
+				if (_mouse_down_item != r)
+					r->select(false);
+
+			if (_mouse_down_item)
+				show_popup(_mouse_down_item);
+		}
 	}
 
-	selection_changed();
-	update();
+	_dragging = false;
 }
 
 void Header::mouseReleaseEvent(QMouseEvent *event)
 {
 	assert(event);
-	if (event->button() == Qt::LeftButton) {
-		if (_dragging)
-			_view.normalize_layout();
-		else
-		{
-			const shared_ptr<RowItem> mouse_over_row_item =
-				get_mouse_over_row_item(event->pos());
-			if (mouse_over_row_item)
-				show_popup(mouse_over_row_item);
-		}
+	if (event->button() & Qt::LeftButton)
+		mouseLeftReleaseEvent(event);
 
-		_dragging = false;
-		_drag_row_items.clear();
-	}
+	_mouse_down_item = nullptr;
 }
 
 void Header::mouseMoveEvent(QMouseEvent *event)
@@ -214,28 +228,33 @@ void Header::mouseMoveEvent(QMouseEvent *event)
 		QApplication::startDragDistance())
 		return;
 
-	// Move the signals if we are dragging
-	if (!_drag_row_items.empty())
-	{
-		_dragging = true;
+	// Check all the drag items share a common owner
+	RowItemOwner *item_owner = nullptr;
+	for (shared_ptr<RowItem> r : _view)
+		if (r->dragging()) {
+			if (!item_owner)
+				item_owner = r->owner();
+			else if(item_owner != r->owner())
+				return;
+		}
 
-		const int delta = event->pos().y() - _mouse_down_point.y();
+	if (!item_owner)
+		return;
+
+	// Do the drag
+	_dragging = true;
 
-		for (auto i = _drag_row_items.begin();
-			i != _drag_row_items.end(); i++) {
-			const std::shared_ptr<RowItem> row_item((*i).first);
-			if (row_item) {
-				const int y = (*i).second + delta;
-				row_item->set_v_offset(y);
+	const int delta = event->pos().y() - _mouse_down_point.y();
 
-				// Ensure the trace is selected
-				row_item->select();
-			}
-			
+	for (std::shared_ptr<RowItem> r : _view)
+		if (r->dragging()) {
+			r->set_v_offset(r->drag_point().y() + delta);
+
+			// Ensure the trace is selected
+			r->select();
 		}
 
-		signals_moved();
-	}
+	signals_moved();
 
 	update();
 }