X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fviews%2Ftrace%2Fviewwidget.cpp;h=e4de122d9e7ddd8bc6f0efc5763ac3b338fc63c2;hp=d5b949c0dabcf89e196dadeeb35bf37a230c2cf2;hb=9f094349d415a37ec30fa2aa2bb6c979c03d6bc1;hpb=f4ab4b5c657e5613caba82feaa81a8a400e4f331 diff --git a/pv/views/trace/viewwidget.cpp b/pv/views/trace/viewwidget.cpp index d5b949c0..e4de122d 100644 --- a/pv/views/trace/viewwidget.cpp +++ b/pv/views/trace/viewwidget.cpp @@ -105,9 +105,9 @@ void ViewWidget::drag_items(const QPoint &delta) bool item_dragged = false; // Drag the row items - const vector< shared_ptr > row_items( - view_.list_by_type()); - for (const shared_ptr& r : row_items) + const vector< shared_ptr > row_items( + view_.list_by_type()); + for (const shared_ptr& r : row_items) if (r->dragging()) { r->drag_by(delta); @@ -252,17 +252,25 @@ void ViewWidget::mousePressEvent(QMouseEvent *event) { assert(event); - /* Ignore right click events as they will open context menus when + if (event->button() & Qt::LeftButton) { + if (event->modifiers() & Qt::ShiftModifier) + view_.show_cursors(false); + + mouse_down_point_ = event->pos(); + mouse_down_offset_ = view_.offset() + event->pos().x() * view_.scale(); + mouse_down_item_ = get_mouse_over_item(event->pos()); + mouse_left_press_event(event); + } + + /* Don't forward right click events as they will open context menus when * used on trace labels. Those menus prevent ViewWidget::mouseReleaseEvent() * to be triggered upon button release, making mouse_down_item_ * hold the last reference to a view item that might have been deleted * from the context menu, preventing it from being freed as intended. + * TODO Remove this once context menus are handled separately */ - if (event->button() & Qt::LeftButton) { + if (event->button() & Qt::RightButton) mouse_down_point_ = event->pos(); - mouse_down_item_ = get_mouse_over_item(event->pos()); - mouse_left_press_event(event); - } } void ViewWidget::mouseReleaseEvent(QMouseEvent *event) @@ -276,33 +284,68 @@ void ViewWidget::mouseReleaseEvent(QMouseEvent *event) mouse_down_item_ = nullptr; } +void ViewWidget::keyReleaseEvent(QKeyEvent *event) +{ + // Update mouse_modifiers_ also if modifiers change, but pointer doesn't move + if ((mouse_point_.x() >= 0) && (mouse_point_.y() >= 0)) // mouse is inside + mouse_modifiers_ = event->modifiers(); + update(); +} + +void ViewWidget::keyPressEvent(QKeyEvent *event) +{ + // Update mouse_modifiers_ also if modifiers change, but pointer doesn't move + if ((mouse_point_.x() >= 0) && (mouse_point_.y() >= 0)) // mouse is inside + mouse_modifiers_ = event->modifiers(); + update(); +} + void ViewWidget::mouseMoveEvent(QMouseEvent *event) { assert(event); mouse_point_ = event->pos(); + mouse_modifiers_ = event->modifiers(); if (!event->buttons()) item_hover(get_mouse_over_item(event->pos()), event->pos()); - else if (event->buttons() & Qt::LeftButton) { - if (!item_dragging_) { - if ((event->pos() - mouse_down_point_).manhattanLength() < - QApplication::startDragDistance()) - return; - if (!accept_drag()) - return; + if (event->buttons() & Qt::LeftButton) { + if (event->modifiers() & Qt::ShiftModifier) { + // Cursor drag + pv::util::Timestamp current_offset = view_.offset() + event->pos().x() * view_.scale(); - item_dragging_ = true; - } + const int drag_distance = qAbs(current_offset.convert_to() - + mouse_down_offset_.convert_to()) / view_.scale(); - // Do the drag - drag_items(event->pos() - mouse_down_point_); + if (drag_distance > QApplication::startDragDistance()) { + view_.show_cursors(true); + view_.set_cursors(mouse_down_offset_, current_offset); + } else + view_.show_cursors(false); + + } else { + if (!item_dragging_) { + if ((event->pos() - mouse_down_point_).manhattanLength() < + QApplication::startDragDistance()) + return; + + if (!accept_drag()) + return; + + item_dragging_ = true; + } + + // Do the drag + drag_items(event->pos() - mouse_down_point_); + } } } void ViewWidget::leaveEvent(QEvent*) { mouse_point_ = QPoint(-1, -1); + mouse_modifiers_ = Qt::NoModifier; + item_hover(nullptr, QPoint()); update(); }