Automatically set foldcolumn #93
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Prelim work on fixing my autofold feature This should probably eventually turn into it's own plugin.
The pattern ?* requires there to be an actual character in the filename, I believe, as it wasn't matching when I do :new. The fix to reset belloff was that I simply didn't understand eval option syntax.
Now it has parameters that control what to set foldcolumn to, and returns the value for you to use rather than setting foldcolumn itself. This could be extended into a plugin Auto Fold Column, with autoloaded functions that detect folds, return the right values (based on global variables, options passed in, and sane defaults), and easy to set up with a simple :call AutoFoldColumn() " Use defaults or global vars :call AutoFoldColumn(col, def) " Use the variables passed in AutoFoldColumn would set up the auto command. Need to google if Vim has a concept of default parameters.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Tweak the autofoldcolumn functionality
Related Issue
None...
Motivation and Context
Solves the problem of beeping unnecessarily and being not modular.
How Has This Been Tested?
✅
Behavior
Open files with folds and without folds, or create and delete them, and watch as the fold column appears and disappears.
Types of changes
Checklist: