New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEATURE][needs-docs] add filter line edit to query builder #6704
Conversation
Any objection to merge this? |
for me no problem, but seems travis is not happy, btw seems not related with your PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good change! Code is certainly much cleaner too
src/gui/qgsquerybuilder.cpp
Outdated
@@ -73,6 +73,9 @@ QgsQueryBuilder::QgsQueryBuilder( QgsVectorLayer *layer, | |||
lblDataUri->setText( tr( "Set provider filter on %1" ).arg( layer->name() ) ); | |||
txtSQL->setText( mOrigSubsetString ); | |||
|
|||
mFilterLineEdit->setPlaceholderText( tr( "Search" ) ); | |||
connect( mFilterLineEdit, &QgsFilterLineEdit::textChanged, this, &QgsQueryBuilder::onTextChanged ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mFilterLineEdit->setShowSearchIcon( true );
(probably needed for the expression query builder too!)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you saying to replace the placeholder string with icon?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I'm saying add it also so it gets the nice search/filter icon too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, done.
Description
Add a filter line edit to filter values in query builder (same as in expression builder widget)
Checklist
fixes #11111
in the commit message next to the description[FEATURE]
in the commit message[needs-docs]
in the commit message and containt sufficient information in the commit message to be documentedscripts/prepare-commit.sh
script before each commit