Skip to content
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

Improve doctex folding #4534

Merged
merged 1 commit into from
Feb 21, 2025
Merged

Improve doctex folding #4534

merged 1 commit into from
Feb 21, 2025

Conversation

jlelong
Copy link
Collaborator

@jlelong jlelong commented Feb 20, 2025

Related to #4527

First, we must keep in mind from the documentation of registerFoldingRangeProvider (see https://code.visualstudio.com/api/references/vscode-api)

If a folding range overlaps with an other range that has a smaller position, it is also ignored.

This PR improves things by properly computing overlapping ranges. This improves folding for the case of #4527 even though not all driver regions will be properly folded because of VS Code's limitation. Yet, the implementation will become fully effective when VS Code removes the overlapping limitation.

@jlelong jlelong requested a review from James-Yu February 20, 2025 15:54
@James-Yu James-Yu merged commit 2874f91 into master Feb 21, 2025
11 checks passed
@James-Yu James-Yu deleted the 4527-dtx-env-folding branch February 21, 2025 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants