Added toggle for displaying symlink destination#1367
Added toggle for displaying symlink destination#1367rzvxa wants to merge 2 commits intopreservim:masterfrom
Conversation
|
@alerque I would appreciate it if you review this one too, It's been hanging out here for over a month. |
alerque
left a comment
There was a problem hiding this comment.
You're welcome to do the honors here and merge. Since it's a single discrete change I think the merge commit from master is kind of irrelevant, but at this point you can just use the "squash and merge" feature in the PR to get the PR merged using one commit to the master branch.
As far as I'm concerned this is good to go, but if you want to add any changelog stuff you can still do that here before merging and still use squash to get it all down to one commit for how it pulls through to most plugin managers...
|
Thanks, I was holding back on merging this one until some feedback came. |
Description of Changes
Added an option to settings for showing or hiding symlink destinations.
Note:
This actually hides the special highlighting for symlink, As its highlighting depends on the
->mark being present and I've tried to reflect that in the documentation.Changing this behavior may need changes to the way symlinks are represented so we can highlight them without the destination path being present.
New Version Info
Author's Instructions
MAJOR.MINOR.PATCHversion number. Increment the:MAJORversion when you make incompatible API changesMINORversion when you add functionality in a backwards-compatible mannerPATCHversion when you make backwards-compatible bug fixesCollaborator's Instructions