Merge pull request #2493 from SebDieBln/AttrTableHideSortIndicator

Hide the sort indicator in the attribute table until first sort (fixes #6705)

Author: m-kuhn

src/gui/attributetable/qgsattributetableview.cpp

@@ -53,12 +53,14 @@ QgsAttributeTableView::QgsAttributeTableView( QWidget *parent )
 
   setSelectionBehavior( QAbstractItemView::SelectRows );
   setSelectionMode( QAbstractItemView::ExtendedSelection );
-  setSortingEnabled( true );
+  setSortingEnabled( true ); // At this point no data is in the model yet, so actually nothing is sorted.
+  horizontalHeader()->setSortIndicatorShown( false ); // So hide the indicator to avoid confusion.
 
   verticalHeader()->viewport()->installEventFilter( this );
 
   connect( verticalHeader(), SIGNAL( sectionPressed( int ) ), this, SLOT( selectRow( int ) ) );
   connect( verticalHeader(), SIGNAL( sectionEntered( int ) ), this, SLOT( _q_selectRow( int ) ) );
+  connect( horizontalHeader(), SIGNAL( sortIndicatorChanged( int, Qt::SortOrder ) ), this, SLOT( showHorizontalSortIndicator() ) );
 }
 
 QgsAttributeTableView::~QgsAttributeTableView()
@@ -282,3 +284,8 @@ void QgsAttributeTableView::selectRow( int row, bool anchor )
       mFeatureSelectionModel->selectFeatures( QItemSelection( tl, br ), command );
   }
 }
+
+void QgsAttributeTableView::showHorizontalSortIndicator()
+{
+  horizontalHeader()->setSortIndicatorShown( true );
+}

src/gui/attributetable/qgsattributetableview.h

@@ -139,6 +139,7 @@ class GUI_EXPORT QgsAttributeTableView : public QTableView
 
   private slots:
     void modelDeleted();
+    void showHorizontalSortIndicator();
 
   private:
     void selectRow( int row, bool anchor );