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

feat(format/html): implement bracketSameLine #4968

Open
wants to merge 1 commit into
base: next
Choose a base branch
from

Conversation

dyc3
Copy link
Contributor

@dyc3 dyc3 commented Jan 24, 2025

Summary

This PR implements functionality for the bracketSameLine option, with a particular focus on matching prettier's emitted IR.

closes #4024

Test Plan

added tests

@dyc3 dyc3 marked this pull request as draft January 24, 2025 17:44
@github-actions github-actions bot added A-Formatter Area: formatter L-HTML Language: HTML labels Jan 24, 2025
write!(f, [soft_line_break_or_space()])?;
}

// TODO: These tokens (the `/>`) are not yet borrowed by sibling elements for whitespace sensitivity.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whitespace sensitivity is not yet implemented for self closing elements

Copy link

codspeed-hq bot commented Jan 24, 2025

CodSpeed Performance Report

Merging #4968 will not alter performance

Comparing html-bracket-same-line (82f9a47) with next (d76e8d3)

Summary

✅ 95 untouched benchmarks

@dyc3 dyc3 force-pushed the html-bracket-same-line branch 2 times, most recently from e583d33 to 8afc1a8 Compare January 27, 2025 14:13
@dyc3 dyc3 marked this pull request as ready for review January 27, 2025 14:40
@dyc3 dyc3 force-pushed the html-bracket-same-line branch from 8afc1a8 to b27188e Compare January 27, 2025 15:01
@dyc3

This comment was marked as resolved.

@dyc3 dyc3 marked this pull request as draft January 27, 2025 15:55
@dyc3 dyc3 force-pushed the html-bracket-same-line branch from b27188e to 82f9a47 Compare January 29, 2025 12:44
@dyc3 dyc3 marked this pull request as ready for review January 29, 2025 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Formatter Area: formatter L-HTML Language: HTML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant