[FEAT] Wildcards or command substitution in hideFolders in the disk module

[NorseJedi]

Description

It would be nice to be able to hide all mounted disks under a specific directory, like /mnt/wsl/docker-desktop-bind-mounts/* on Windows/WSL which tend to have multiple very long paths with random names (see screenshot).

I read that the source-property for the logo module supports command substitution, which would do the trick also in the hideFolder disk module, but it doesn't seem to be supported there.

So it would be very nice to be able to either specify a folder wildcard like /mnt/wsl/* or *docker-desktop* to hide all mountpoints it matches, or alternatively be able to use command substitution so putting in something like $(mount | grep 'bind-mounts' | awk '{ print $3 }' | tr '\n' ':'):/efi:/boot/efi:/boot/firmware would work (or if combining substitution and static entries is overly complicated, allowing the property to only be set to $(mount | grep -E 'bind-mounts|boot|efi' | awk '{ print $3 }' | tr '\n' ':') could do the job).

Motivation

It's a request for an improvement in an existing module, I honestly haven't even thought of recreating this functionality using a different module.

Additional context

[CarterLi]

Instead of using hideFolders, you may use "folders": "/:/mnt/home".

specify a folder wildcard like /mnt/wsl/* or *docker-desktop* to hide all mountpoints it matches

It requires regexp library or glob library.

command substitution so putting in something like $(mount | grep 'bind-mounts' | awk '{ print $3 }' | tr '\n' ':'):/efi:/boot/efi:/boot/firmware would work

It won't work. Besides, I don't want to make the implementation of this seldom-used feature too complex.

[NorseJedi]

Instead of using hideFolders, you may use "folders": "/:/mnt/home".

I know, but that presents a similar problem in my case. I have many other computers, all with different mountpoints (some with quite many), but I have a common dotfiles setup managed with git and stow that I use on all of them, and this includes my fastfetch config-file.

It requires regexp library or glob library.

Yes, I suppose. You don't want to add dependencies, is that it? I can understand that.

It won't work. Besides, I don't want to make the implementation of this seldom-used feature too complex.

I just assumed it would be a trivial thing to implement since it's already available for logos. But if you don't want to, then that's your prerogative. Perhaps if this feature was available, it wouldn't be so seldomly used? ;) Anyway, I hope you'll reconsider, this would be - as far as I can tell - a very small modification that has no drawbacks while it adds flexibility for the user, which in my opinion is always preferable in software.

If you want I can probably implement it myself and submit a pull request, but since I'm not familiar with the codebase it would take a bit of time and effort. I'll gladly do it, but not if you've already decided you won't merge it.

[NorseJedi]

Very much appreciated, thank you :)