What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This PR fixes #159 where multiple regions would not correctly fold in vscode. This PR also bumps the verison number and dependencies using npm update
What is the current behavior? (You can also link to an open issue here)
Currently the folder directives # region and # endRegion only work for a single instance. With multiple folding regions this does not work as intended.
What is the new behavior (if this is a feature change)?
Folding regions work as expected.
Other information:
The underlying issue was an implicit dependencies on the folding regions being in order relative to their appearance in the file. The directive parser contains individual fields for region, endRegion and include. In kls these are passed to the folding service as follows.
This would effectively place all regions first followed by all endregion. This lead to a single region working but multiple failing. The fix was to sort regions using the range based comparer.
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) This PR fixes #159 where multiple regions would not correctly fold in vscode. This PR also bumps the verison number and dependencies using
npm update
What is the current behavior? (You can also link to an open issue here) Currently the folder directives
# region
and# endRegion
only work for a single instance. With multiple folding regions this does not work as intended.What is the new behavior (if this is a feature change)? Folding regions work as expected.
Other information: The underlying issue was an implicit dependencies on the folding regions being in order relative to their appearance in the file. The directive parser contains individual fields for
region
,endRegion
andinclude
. Inkls
these are passed to the folding service as follows.This would effectively place all regions first followed by all endregion. This lead to a single region working but multiple failing. The fix was to sort regions using the range based comparer.