Skip to content

Conversation

@qtyosaf
Copy link
Contributor

@qtyosaf qtyosaf commented Feb 15, 2023

Changes:
Color named FolderSymlink added in colors.lua.
Color applied to folder symlinks in _build_folder function in builder.lua
Added color to Highlight Groups in documentation.

I believe this change is necessary to add more contrast for folder symlinks.

Copy link
Member

@alex-courtis alex-courtis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works and will be useful, thanks for your contribution.

Please:

  • remove defaults, linking to FolderName

@qtyosaf qtyosaf force-pushed the folder-symlink-colors branch from f4466f9 to 68676cf Compare April 3, 2023 17:20
@qtyosaf
Copy link
Contributor Author

qtyosaf commented Apr 3, 2023

Sorry for taking so long to update the pull request! I changed things according to the comments.

Changes:
Redid the _build_folder function so that checks for being a symlink only happen once.
No default color for symlink folders, links to Directory.
Color name changed to NvimTreeSymlinkFolderName for parity.

Copy link
Member

@alex-courtis alex-courtis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested:

  • default to Directory
  • default to NvimTreeFolderName
  • uses NvimTreeSymlinkFolderName

@alex-courtis alex-courtis changed the title Added FolderSymlink color that is applied in builder.lua feat(renderer): add NvimTreeSymlinkFolderName Apr 6, 2023
@alex-courtis alex-courtis merged commit f3dbddf into nvim-tree:master Apr 15, 2023
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