Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add file explorer #11285
Add file explorer #11285
Changes from all commits
b1dcd96
bb1722a
02edda1
7902fc5
268eac8
ff833b7
006359c
1233784
0b01885
52a9cef
996ee93
11435df
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't call this an error, honestly. For example I'm frequently doing
hx .
and then opening the file browser, since i am in a[scratch]
, everytime I do that it adds a red message to the statusline, so it looks like something went wrong, but really it's just fineI think that this call to
set_error
can just be removedThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be fair, this function is called
file_browser_in_current_buffer_directory
, so this error is quite justified to inform the user that the[scratch]
buffer does not have a dedicated directory. In my opinion, it would be confusing to remove this message for cases when something actually goes wrong.What you want to call after
hx .
is probablyfile_browser_in_current_directory
or simplyfile_browser
. Now I agree, that it is more convenient to have a single key combination that covers all file browsing needs in one go, and arguablyfile_browser_in_current_buffer_directory
already does that for the price of showing you a somewhat-irritating-but-easily-ignorable error message. In an ideal world one would probably have to wait for when programmable config will be available in helix and code the desired behavior themself.