Ability to not fold function names and docstrings#114
Open
siddarthio wants to merge 3 commits into
Open
Conversation
Added two variables which allow users not to fold function names (à la foldmethod=indent) and also allow the docstrings not to be folded. There are a couple of considerations I should put into the README, which are both new variables need default values, and also if you want to use unfold_docstring, unfold_function name should also be set to enable.
Added the new variables and the issues of the unfold_docstring variable.
|
Fantastic plugin to use, this feature is something I would appreciate. Thanks. |
spire-carlos
suggested changes
Aug 10, 2021
|
|
||
| To work properly, it needs `SimpylFold_fold_docstring` to be set to the | ||
| default value `1` to find the docstring starting location. In addition, | ||
| `SimpylFold_fold_docstring` overrides the affect of |
There was a problem hiding this comment.
Suggested change
| `SimpylFold_fold_docstring` overrides the affect of | |
| `SimpylFold_fold_docstring` overrides the effect of |
Typo
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
I wanted the ability to see function names not be incorporated into the fold, similar to (Issue: folds the function names too? #108) and also I saw that a lot of people wanted the ability to stop Docstrings from being folded (Issue: Option to start definition fold after docstring #84). So I added both of those.
I just started using Vim about a week ago, so I wasn't able to find out how to change the default value of the variables to 0. I think that this should be done, otherwise when people run
PlugUpdateor its equivalent, they will need to change their .vimrc.Thanks for Plugin btw, I absolutely love it, It's way better than the shitty folding that Atom had.