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

diff --git a/pv/view/header.cpp b/pv/view/header.cpp
index 202ae10..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);
+
+	const bool ctrl_pressed =
+		QApplication::keyboardModifiers() & Qt::ControlModifier;
 
-	// Select the signal if it has been clicked
-	const shared_ptr<RowItem> mouse_over_row_item =
+	// 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,37 +228,31 @@ void Header::mouseMoveEvent(QMouseEvent *event)
 		QApplication::startDragDistance())
 		return;
 
-	// Check the list of dragging items is not empty
-	if (_drag_row_items.empty())
-		return;
-
 	// Check all the drag items share a common owner
-	const shared_ptr<RowItem> first_row_item(
-		_drag_row_items.front().first);
-	for (const auto &r : _drag_row_items) {
-		const shared_ptr<RowItem> row_item(r.first);
-		assert(row_item);
-
-		if (row_item->owner() != first_row_item->owner())
-			return;
-	}
+	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;
+		}
+
+	if (!item_owner)
+		return;
 
 	// Do the drag
 	_dragging = true;
 
 	const int delta = event->pos().y() - _mouse_down_point.y();
 
-	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);
+	for (std::shared_ptr<RowItem> r : _view)
+		if (r->dragging()) {
+			r->set_v_offset(r->drag_point().y() + delta);
 
 			// Ensure the trace is selected
-			row_item->select();
+			r->select();
 		}
-	}
 
 	signals_moved();