-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow specifying a number of context lines per entry that should not be folded #4
Comments
closed with e3e0076 Regions that contain just strings will not be folded.(I try to avoid adding new settings) |
one additional thing, this plugin has no longer build in rules for tailwind classes to fold, |
Hm, I don't really like this, unfortunately. The whitespace between the beginning of the string and the summary text on the second line leads to one unnecessary line being displayed. I don't really see a way to go about this other than introducing a new setting, but instead of setting an arbitrary number of context lines (that will work for some cases but won't for others), perhaps it is possible to split the rules in a way that a user can select a region that should be considered for folding in general (here: Just a quick idea off my head, though, as the code probably would probably become less elegant that way. Another alternative would be a separate "disable for expanded selection" setting or similar that utilizes |
With the changes to the Python default package, it is now possible to fold the non-summary parts of a docstring, i.e. everything but the first line. However, when the summary line is on the second line, it means that we have folded regions on both sides of a line that include newlines, which makes for a somewhat whacky experience when navigating the caret within the docstring. See the apng in the following PR for an example:
deathaxe/sublime-packages#2 (comment)
You can reproduce this by using the Python package from the Package's
master
branch and the following config:My proposal for this situation would be to add a configuration for context line where folding would not be applied for regions that are within n lines of the caret instead of on the same line as the caret.
Another idea related to this would be to unfold any regions that share a line with the currently active region (i.e. a region that would be folded if the caret wasn't inside it) and/or consider the additional config option here.
The text was updated successfully, but these errors were encountered: