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

independent hidden option for file/folder browser #227

Closed
jamestrew opened this issue Feb 11, 2023 · 1 comment · Fixed by #268
Closed

independent hidden option for file/folder browser #227

jamestrew opened this issue Feb 11, 2023 · 1 comment · Fixed by #268
Assignees
Labels
enhancement New feature or request

Comments

@jamestrew
Copy link
Collaborator

Leaving this mostly for myself as a reminder.

Currently the hidden option applies to both the file & folder browser. I'd like to have the ability to have something like hidden = true for the file browser but hidden = false for the folder browser.

I think turning the hidden option to be able to optionally take a table will be a fairly simple way to accommodate this.
eg. hidden = { file_browser = true, folder_browser = false }.
With this, I'm leaning on changing the default to hidden = { file_browser = false, folder_browser = false }. This will preserve the default behavior but illustrate the capacity to differentiate the hidden option between the browsers.

@jamestrew jamestrew added the enhancement New feature or request label Feb 11, 2023
@jamestrew jamestrew self-assigned this Feb 11, 2023
@fdschmidt93
Copy link
Member

With this, I'm leaning on changing the default to hidden = { file_browser = false, folder_browser = false }

#210 has pretty much exactly this in broader scope (ie any option). Users will be able to define their own pickers. I just need the time and motivation to finish it 😅

jamestrew added a commit that referenced this issue May 6, 2023
jamestrew added a commit that referenced this issue May 6, 2023
* feat: separate hidden opts for file/folder browser

closes #227

* [docgen] Update doc/telescope-file-browser.txt
skip-checks: true

---------

Co-authored-by: Github Actions <actions@github>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants