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

Allow customizing labels #23

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

Allow customizing labels #23

wants to merge 11 commits into from

Conversation

eugenesvk
Copy link
Contributor

@eugenesvk eugenesvk commented Aug 13, 2021

Think this is the last config option I wanted to add — for the labels themselves

Also finally fixed that nasty ValueError bug! Don't understand why, but importing and calling the core.commands functions directly seemed to do the trick (alternative that worked: copy one more command InitHiddenFilesFilter from Core and call it ourselves, but that would require disabling it in the Core (last time I tried it without disabling the Core command the pane wouldn't show any files at all as the filters seemed to just duplicate)
there might be something in the closed source pane._remove_filter that leads to this or maybe just some general object thingy I don't get, but anyway, but glad it's sorted out ✨✨! (even though we need to use a private Core function)

FYI: since each bug fix is structured as a minor version bump, don't forget to make 4 tags/releases as well if you want the links to work correctly

v0.4.4 — 15.08.2021

  • Added
    • HideDotfile now also affects the pane list, not just the status bar

v0.4.3 — 14.08.2021

  • Fixed
    • 🪲 ValueError on the first Toggle hidden files if a pane is launched with hidden files hidden

v0.4.2 — 14.08.2021

  • Fixed
    • 🪲 glob meta-characters [ ] * ? in paths (e.g. [A--_B]) crash glob or lead to wrong results

v0.4.1 — 13.08.2021

  • Added
    • ✨ User-configurable options for the Folder/File/Size labels

      Option Default Description
      Label Dirs: Files: Size: Folder/File/Size labels
      Hide0Label True Hide labels when 0 folders/files

Escape meta-characters * ? [ ] by [bracketing them]

Fixes: kek91#22
… hidden

For some reason you need to call the core.commands functions instead of their copies otherwise the setting the filter on launch wasn't recognized by them and resulted in a ValueError
@kek91
Copy link
Owner

kek91 commented Aug 16, 2021

Thanks! Don't have time at the moment, but will check it out later tonight.

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.

2 participants