Fix copy items from value map configuration

QAbstractItemView internally uses the KeyPress signal, so in order to make sure our code takes precedence we need to react to the same signal. Apparently it's not enough to trust Qt that the KeyReleased is always triggered after a KeyPress. Fix #32256
qgis · Oct 30, 2019 · 9bf92ef · 9bf92ef
1 parent bcf2040
commit 9bf92ef
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/gui/editorwidgets/qgsvaluemapconfigdlg.cpp b/src/gui/editorwidgets/qgsvaluemapconfigdlg.cpp
@@ -202,12 +202,13 @@ void QgsValueMapConfigDlg::populateComboBox( QComboBox *comboBox, const QVariant
 bool QgsValueMapConfigDlg::eventFilter( QObject *watched, QEvent *event )
 {
   Q_UNUSED( watched )
-  if ( event->type() == QEvent::KeyRelease )
+  if ( event->type() == QEvent::KeyPress )
   {
     QKeyEvent *keyEvent = static_cast<QKeyEvent *>( event );
     if ( keyEvent->matches( QKeySequence::Copy ) )
     {
       copySelectionToClipboard();
+      event->accept();
       return true;
     }
   }