-
-
Notifications
You must be signed in to change notification settings - Fork 639
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
feat(#195): display current path in the strider #1681
base: main
Are you sure you want to change the base?
feat(#195): display current path in the strider #1681
Conversation
Refs: zellij-org#195 Display current path on top of the directory files in the strider default plugin. As of now, no configuration is available for this feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for picking this up!
I gave it a quick run and found two small things to adjust :)
Co-authored-by: Thomas Linford <tlinford@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let current_path = name.clone(); | ||
FsEntry::display_with_shortened_name_start(current_path, width) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should just return empty string? just did another run and i'm seeing extra output on the right
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, thanks for pointing this out. I will check this today in the evening.
Hey, what's the status here? |
Want to echo @imsnif , what's the status on this? This is an important feature that makes the file tree more useful, unless there's another way to switch dir within the strider window. |
Looks like this PR can be closed, as it's functionality has been implemented in #3200. |
This pull request addresses the #195 issue and resolves the problem of @RobWalt's solution with clicks 1-row offset.
As in Rob's PR, the path displayed is a relative path from the
zellij
's start location.Display the current path on top of the directory files in the strider default plugin. As of now, no configuration
is available for this feature.
Examples of how it looks: