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

#837 search subdirs #1092

Merged
merged 1 commit into from Mar 20, 2022
Merged

#837 search subdirs #1092

merged 1 commit into from Mar 20, 2022

Conversation

alex-courtis
Copy link
Member

@alex-courtis alex-courtis commented Mar 20, 2022

It is possible that I am missing the point, however it seems to me that search is not working.

Changes:

  • rewrote to use filesystem search; node search was problematic and complex
  • matches input that exactly matches the end of a path e.g. file1 dir1/ dir1/file11 but not ir1/file11 or le11
  • adds :find style command line completion for all files under cwd

Resolves:

  • bug in which searching would result in two nodes would be created for the same file

Does not:

  • search under the current node; searches under cwd; this could be added

Exposed:

Copy link
Member

@kyazdani42 kyazdani42 left a comment

Choose a reason for hiding this comment

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

indeed seemed like it wasn't working. Good idea to fallback to find_file

@kyazdani42 kyazdani42 merged commit fc70954 into master Mar 20, 2022
@be-ndee
Copy link
Contributor

be-ndee commented Mar 22, 2022

Okay, maybe I gave the action a miss leading name :-D A better name would have been "go to file". So my intention was that you can enter a path to jump to that directory/file in the tree. So you don't have to click or navigate through the whole tree. But "your" search is also a good feature. But it would be good, that the search checks the options for ignored files. Because I have a node_modules directory, which is in the gitignore and not visible for me in the tree, but the search searches in the node_modules, so it's quite slow.

@kyazdani42
Copy link
Member

why not just using / then ? I don't understand the point of just moving the cursor to a file ? The search should not search inside ignored directories right ?

@alex-courtis
Copy link
Member Author

Raised #1099

2 seems like a workable solution, however I have a feeling that there is something simpler.

@alex-courtis
Copy link
Member Author

alex-courtis commented Mar 23, 2022

why not just using / then

That matched the functionality of S when I was testing. Finding files by name in subdirectories was not working.

It seems that I was misunderstanding the purpose of S. Please accept my apologies @be-ndee

@kyazdani42 kyazdani42 deleted the #837-search-subdirs branch April 30, 2022 12:33
Almo7aya pushed a commit to Almo7aya/nvim-tree.lua that referenced this pull request Oct 11, 2022
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.

None yet

3 participants