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

Included end tag in HTML folding range #127

Closed

Conversation

AlexXuChen
Copy link

Included end tag in HTML folding range

Fixes #126

Signed-off-by: Alexander Chen [email protected]

@ghost
Copy link

ghost commented Apr 4, 2022

CLA assistant check
All CLA requirements met.

@aeschli
Copy link
Contributor

aeschli commented Apr 8, 2022

We'd need an option for that as we don't want to lose the existing behavior

What if there's more content after the closing tag?

@aeschli aeschli self-assigned this Apr 8, 2022
@aeschli aeschli self-requested a review April 8, 2022 07:44
@AlexXuChen
Copy link
Author

How would one go about adding a setting to this? Would this be on the vscode client side or added here?

@AlexXuChen AlexXuChen force-pushed the folding-range-with-end-tag branch from 861f79c to 59a19cc Compare April 14, 2022 17:02
@aeschli
Copy link
Contributor

aeschli commented Apr 25, 2022

Yes there's also a new setting needed in the JSON language extension. That is in https://github.com/microsoft/vscode.

Can you add a test case where the closing tag is on the same line as the next opening tag?

@AlexXuChen AlexXuChen force-pushed the folding-range-with-end-tag branch from 59a19cc to afed461 Compare April 26, 2022 19:59
@aeschli
Copy link
Contributor

aeschli commented Sep 8, 2023

closing, PR hasn't been worked on for a while

@aeschli aeschli closed this Sep 8, 2023
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.

HTML folding range should include end tag
2 participants