Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork97
Preserve whitespace for pre/textarea when pretty-printing#432
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Preserve whitespace for pre/textarea when pretty-printing#432
Uh oh!
There was an error while loading.Please reload this page.
Conversation
changeset-botbot commentedSep 30, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
🦋 Changeset detectedLatest commit:96dfd13 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means?Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Seems reasonable but I'm a bit worried this might break folks using pretty printing in their test suites and the like. Pretty specific tags & change scenarios though, so perhaps the risk is low
Uh oh!
There was an error while loading.Please reload this page.
69ed337 intopreactjs:mainUh oh!
There was an error while loading.Please reload this page.
<pre>and<textarea>are whitespace-sensitive. When pretty-printing, it's important to not inject extra whitespace when rendering them.There may be more tags for which whitespace should be preserved; these are the two that I specifically know about.
Disclosure: I wrote the tests by hand and used GPT-5-Codex to help with the implementation.