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

User selected columns to filter #817

Closed
wants to merge 3 commits into from
Closed

Conversation

MaksimProgr
Copy link
Contributor

For the issue: #747
I have done this by splitting and then moving indexed parts from anyMatchFilter to filters array. Is it an appropriate solution?

@Mottie
Copy link
Owner

Mottie commented Feb 11, 2015

Hi @MaksimProgr!

Thanks for sharing this! I'll try it out when I get some time.

I was actually asking for opinions in that issue on a good straight-forward method that makes sense. I'll rustle up some feed back from others and see what the consensus is on this method.

Thanks again!

@Mottie
Copy link
Owner

Mottie commented Feb 21, 2015

I'm merged your changes in e6db1fa.

I also made a few modifications, including adding a filter_columnAnyMatch option which allow a developer to disable the column search - see c0911bc

Please note that the filter widget was moved to a separate file (widget-filter.js) when I added a grunt build process.

Thanks again for your contribution!

@Mottie Mottie closed this Feb 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants