[ui] Fix unmet expectation: when navigating the browser panel via key…

…board, hitting return/enter should load focused dataset
qgis · Oct 11, 2021 · f774de9 · f774de9
1 parent 768b69c
commit f774de9
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 0 deletions.
diff --git a/python/gui/auto_generated/qgsbrowsertreeview.sip.in b/python/gui/auto_generated/qgsbrowsertreeview.sip.in
@@ -48,6 +48,11 @@ Returns the browser model
 
     void setSettingsSection( const QString &section );
 
+  protected:
+
+    virtual void keyPressEvent( QKeyEvent *event );
+
+
   protected slots:
     virtual void rowsInserted( const QModelIndex &parentIndex, int start, int end );
 

diff --git a/src/gui/qgsbrowsertreeview.cpp b/src/gui/qgsbrowsertreeview.cpp
@@ -20,6 +20,8 @@
 #include "qgsguiutils.h"
 #include "qgsdataitem.h"
 
+#include <QKeyEvent>
+
 QgsBrowserTreeView::QgsBrowserTreeView( QWidget *parent )
   : QTreeView( parent )
   , mSettingsSection( QStringLiteral( "browser" ) )
@@ -28,6 +30,14 @@ QgsBrowserTreeView::QgsBrowserTreeView( QWidget *parent )
   setIndentation( QgsGuiUtils::scaleIconSize( 16 ) );
 }
 
+void QgsBrowserTreeView::keyPressEvent( QKeyEvent *event )
+{
+  if ( event->key() == Qt::Key_Return || event->key() == Qt::Key_Enter )
+    emit doubleClicked( currentIndex() );
+  else
+    QTreeView::keyPressEvent( event );
+}
+
 void QgsBrowserTreeView::setModel( QAbstractItemModel *model )
 {
 

diff --git a/src/gui/qgsbrowsertreeview.h b/src/gui/qgsbrowsertreeview.h
@@ -51,6 +51,10 @@ class GUI_EXPORT QgsBrowserTreeView : public QTreeView
     // Set section where to store settings (because we have 2 browser dock widgets)
     void setSettingsSection( const QString &section ) { mSettingsSection = section; }
 
+  protected:
+
+    void keyPressEvent( QKeyEvent *event ) override;
+
   protected slots:
     void rowsInserted( const QModelIndex &parentIndex, int start, int end ) override;