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

[tree-filter] Make search case insensitve and accept RegExp #948

Merged
merged 6 commits into from
May 15, 2017

Conversation

wikiped
Copy link
Contributor

@wikiped wikiped commented Apr 6, 2017

Currently filtering entries in the file tree is Case Sensitive. This patch will make it case insensitive and it will also accept Regular Expressions as valid search strings.

@wikiped wikiped changed the title tree-filter: Make search case insensitve and accept RegExp [tree-filter] Make search case insensitve and accept RegExp Apr 6, 2017
@jcb91
Copy link
Member

jcb91 commented Apr 6, 2017

I think there's an issue requesting case sensitivity on the upstream repo https://github.com/jdfreder/tree-filter so it might be worth adding this there too? It'd also be good to either make regex optional, or at least a little more obvious that it's happening (e.g. readme, tooltips)

@juhasch
Copy link
Member

juhasch commented Apr 6, 2017

I think making the regex configurable sounds good.

@wikiped
Copy link
Contributor Author

wikiped commented May 12, 2017

The patch was added to make case sensitivity / RegExp optional.

@jcb91 jcb91 merged commit 0fdebe0 into ipython-contrib:master May 15, 2017
@juhasch
Copy link
Member

juhasch commented May 16, 2017

Thanks @jcb91

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.

3 participants