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

fix: starting telescope or grug-far when hovering a directory #388

Merged
merged 1 commit into from
Aug 18, 2024

Conversation

mikavilpas
Copy link
Owner

Issue:
Given the following directory structure:

  • root/
    • dir1/
      • file1
    • file2

When hovering dir1/ and starting the search or replace operation, the operation would only consider the files in dir1/. This is not what the user would expect. The user would expect the operation to consider the files in root/.

Solution:
When hovering a directory, we should consider the directory the hovered directory is in. This is the same behavior as when hovering a file.

Issue:
Given the following directory structure:

- root/
  - dir1/
    - file1
  - file2

When hovering `dir1/` and starting the search or replace operation, the
operation would only consider the files in `dir1/`. This is not what the
user would expect. The user would expect the operation to consider the
files in `root/`.

Solution:
When hovering a directory, we should consider the directory the hovered
directory is in. This is the same behavior as when hovering a file.
@mikavilpas mikavilpas merged commit d6da015 into main Aug 18, 2024
14 checks passed
@mikavilpas mikavilpas deleted the search-in-directory branch August 18, 2024 07:38
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.

1 participant