Only emit the clicked() signal from views if the left button is used.

This is consistent with QAbstractButton, QCalendarWidget,
QDialogButtonBox and QGroupBox (ie, all other widgets with
a clicked signal)

Task-number: QTBUG-26105

Change-Id: Ieafe988b5c03216796b69a7cd70ac1a03fc12b0a
Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
Reviewed-by: Olivier Goffart <ogoffart@woboq.com>

1 files changed, 3 insertions, 2 deletions

diff --git a/src/widgets/itemviews/qabstractitemview.cpp b/src/widgets/itemviews/qabstractitemview.cpp
index 40aa429260..929f7db406 100644
--- a/src/widgets/itemviews/qabstractitemview.cpp
+++ b/src/widgets/itemviews/qabstractitemview.cpp
@@ -522,7 +522,7 @@ void QAbstractItemViewPrivate::_q_scrollerStateChanged()
 /*!
     \fn void QAbstractItemView::clicked(const QModelIndex &index)
 
-    This signal is emitted when a mouse button is clicked. The item
+    This signal is emitted when a mouse button is left-clicked. The item
     the mouse was clicked on is specified by \a index. The signal is
     only emitted when the index is valid.
 
@@ -1859,7 +1859,8 @@ void QAbstractItemView::mouseReleaseEvent(QMouseEvent *event)
     setState(NoState);
 
     if (click) {
-        emit clicked(index);
+        if (event->button() == Qt::LeftButton)
+            emit clicked(index);
         if (edited)
             return;
         QStyleOptionViewItemV4 option = d->viewOptionsV4();