Skip to content
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

Add ability to easily select all for multi-filter #1946

Merged
merged 1 commit into from
Sep 13, 2017
Merged

Add ability to easily select all for multi-filter #1946

merged 1 commit into from
Sep 13, 2017

Conversation

Posnet
Copy link
Contributor

@Posnet Posnet commented Sep 5, 2017

This is in direct response for Issue 608

The multi filter option is useful, but lacking in an easy way to
select all values. I have added in a psudo option '*' that when selected
automatically fills out the mutli-select with all possible filters.

I have also added in a second psudo option '-' which becomes available
if the multi-filter has all possible values selected.
This makes it easy to clear the multi-filter.

As far as I can tell there are no tests for the front end client.
I will add them if you can point me in the right spot of the codebase.

The multi filter option is useful but lacking in an easy easy way to
select all values. I have added in a psudo option '*' that when selected
automatically fills out the mutli-select with all possible filters.

I have also added in a second psudo option '-' which becomes available
_if_ the multi-filter has all possible values selected.
This makes it easy to clear the multi-filter.
@dimar7
Copy link

dimar7 commented Sep 12, 2017

I updated the file client/app/services/query-result.js and I cannot see the option "*" or "-" in the multi filter.

@Posnet
Copy link
Contributor Author

Posnet commented Sep 12, 2017

@dimar7 how did you update query-result? are you using the webpack dev server or did you recompile the frontend

@dimar7
Copy link

dimar7 commented Sep 12, 2017

yes I did update query result. How can I reccompile the front end? I'm not familiar with webpack dev server,

@dimar7
Copy link

dimar7 commented Sep 12, 2017

How call I the filter using your code?

@arikfr arikfr merged commit e5cbdf3 into getredash:master Sep 13, 2017
@arikfr
Copy link
Member

arikfr commented Sep 13, 2017

👏 well done. You made a lot of people happy now :)

@dimar7
Copy link

dimar7 commented Sep 13, 2017

can anyone say me how can I set de filter with '*' or '-'?

@MinalcarEU
Copy link

+1 Would also like to know how I can add this functionality to the AMI Amazon build!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants