Skip to content

Commit

Permalink
Make Select by Form default instead of By Expression
Browse files Browse the repository at this point in the history
Since it's more user-friendly for beginners
  • Loading branch information
nyalldawson committed Dec 27, 2016
1 parent 1f37238 commit 14f18bd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/app/qgisapp.cpp
Expand Up @@ -2112,10 +2112,10 @@ void QgisApp::createToolBars()
QToolButton *bt = new QToolButton( mAttributesToolBar );
bt->setPopupMode( QToolButton::MenuButtonPopup );
QList<QAction*> selectActions;
selectActions << mActionSelectByExpression << mActionSelectByForm << mActionSelectAll
selectActions << mActionSelectByForm << mActionSelectByExpression << mActionSelectAll
<< mActionInvertSelection;
bt->addActions( selectActions );
bt->setDefaultAction( mActionSelectByExpression );
bt->setDefaultAction( mActionSelectByForm );
QAction* selectionAction = mAttributesToolBar->insertWidget( mActionDeselectAll, bt );
selectionAction->setObjectName( QStringLiteral( "ActionSelection" ) );

Expand Down
2 changes: 1 addition & 1 deletion src/ui/qgisapp.ui
Expand Up @@ -72,8 +72,8 @@
<addaction name="mActionSelectPolygon"/>
<addaction name="mActionSelectFreehand"/>
<addaction name="mActionSelectRadius"/>
<addaction name="mActionSelectByExpression"/>
<addaction name="mActionSelectByForm"/>
<addaction name="mActionSelectByExpression"/>
<addaction name="mActionDeselectAll"/>
<addaction name="mActionSelectAll"/>
<addaction name="mActionInvertSelection"/>
Expand Down

1 comment on commit 14f18bd

@nirvn
Copy link
Contributor

@nirvn nirvn commented on 14f18bd Dec 27, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent. What about giving select by expression a keyboard shortcut similar select by form? Ctrl+F3?

Please sign in to comment.