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

"Filter hidden files" does not hide hidden directories #1082

Closed
NyxCode opened this issue Apr 13, 2023 · 1 comment · Fixed by #1083
Closed

"Filter hidden files" does not hide hidden directories #1082

NyxCode opened this issue Apr 13, 2023 · 1 comment · Fixed by #1083
Labels
GH - Bug Something isn't working
Milestone

Comments

@NyxCode
Copy link

NyxCode commented Apr 13, 2023

Fluidd Version

1.23.4

Browser

Chrome

Device

Desktop PC

Operating System

Windows

What happened

"Filter hidden files" does not hide hidden directories.
I back up my config files in a git repository, so there's a .git directory, which is always shown, regardless of the "Filter hidden files" setting.

What did you expect to happen

Hidden directories should be, well, hidden if "Filter hidden files" is on.

How to reproduce

  1. Create a hidden directory (mkdir .test) in your config directory
  2. Go to the fluidd file browser, turn "Filter hidden files on"

Additional information

No response

@NyxCode NyxCode added GH - Bug Something isn't working GH - Evaluation Needed This topic needs to be discussed to evaluate it's aspects and feasability labels Apr 13, 2023
@pedrolamas pedrolamas removed the GH - Evaluation Needed This topic needs to be discussed to evaluate it's aspects and feasability label Apr 13, 2023
@pedrolamas
Copy link
Member

Hi @NyxCode, thanks for reporting this!

Indeed the filters have been implemented over files only, but I do see your point on having this on folders... we will make the change ASAP! 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GH - Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants