Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Hide zipped(binary) filenames from quick open dialog #4382

Closed
wants to merge 2 commits into from

Conversation

shevchuk
Copy link
Contributor

@shevchuk shevchuk commented Jul 8, 2013

It is sometimes hard to choose specific file using quick open because there are minified zipped files among the list.

} else {
return true;
}
}
Copy link
Member

Choose a reason for hiding this comment

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

This should probably live in ProjectManager so it can sit next to ProjectManager.shouldShow()...

Both these file-typing utility methods should be on a similar path of evolution, something like this:

  • Initially: Easily patchable & reusable
  • Later: Extensible
  • Later still: User-editable (i.e. make it a preference)

Copy link
Member

Choose a reason for hiding this comment

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

One other way that this and shouldShow() don't parallel each other is that shouldShow() uses a regexp. I think that's ok for now (in fact, shouldShow() used an array too up until 8d40193, and I'm guessing it will need to switch back eventually when the list becomes less hardcoded...).

@ghost ghost assigned peterflynn Jul 9, 2013
@peterflynn
Copy link
Member

@shevchuk done reviewing. Thanks for signing the CLA, too!

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

Successfully merging this pull request may close these issues.

2 participants