fix: absolute path support in the symbol browser

psprint commented 1 year ago

Description

Currently the absolute path from TAGS will be prepended by the wrapping git repository path, yielding something like /home/q/cmds//home/q/.local/share/zinit/plugins/zservices---command-server/libexec/cmd-cat which will not work

Motivation and Context

To open files found via the tag menu, prepending with a PWD/git-repo path only when needed.

Related Issue(s)

Usage examples

How Has This Been Tested?

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

psprint commented 1 year ago

@vladdoster I've also optimized the menu in new commit.

psprint commented 1 year ago

PS. Also a new feature – sort toggling via alt-s.

psprint commented 1 year ago

@vladdoster Could this fix be merged? I've updated the commit message to pass linter (added 'fix: …' prefix).

psprint commented 1 year ago

@vladdoster This commit is important because it fixes annoying bug in the browser. I've just fixed commit message.

zdharma-continuum / zinit