Skip to content

Add option to allow expanders in empty folders #3593

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jralo7
Copy link

@jralo7 jralo7 commented Jul 31, 2025

When issue #344 (to hide expanders for empty folders in list mode, for aesthetics) was implemented, it did not only hide the arrows but also disable expanding them altogether. That caused several problems, including the issues #486, #508, #616, #894 and #3145.

This pull request adds an option to toggle #334's behaviour (optional, by default my changes are disabled).

I think a better solution would be to change the expander icons on empty folders, like or instead of hiding the . But that would require modifying a lot more stuff, and my attempts have been unsuccessful.

Disclaimers:

  • I did use AI, but I reviewed the code and now I'm using it on my computer to see if there are any bugs. Once I find no bugs in some time I'll change "Draft PR" to "Open PR", but feel free to test and merge the draft if you want. I haven't experienced any bugs during a few weeks using it on my computer.
  • I closed the previous PR because I messed up my commits while trying to implement the better solution, this PR / branch has cleaner commits.

@jralo7 jralo7 marked this pull request as ready for review August 13, 2025 07:06
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