Skip to content

fix: also escape \ when pretty printing anonymous nodes#14819

Merged
the-mikedavis merged 1 commit intohelix-editor:masterfrom
m4rch3n1ng:tree-escape-backslash
Nov 18, 2025
Merged

fix: also escape \ when pretty printing anonymous nodes#14819
the-mikedavis merged 1 commit intohelix-editor:masterfrom
m4rch3n1ng:tree-escape-backslash

Conversation

@m4rch3n1ng
Copy link
Copy Markdown
Contributor

previously, only double quotes were escaped when pretty printing anonymous nodes, leading to trees like this:

image

this is very easily fixed by just also escaping backslashes:

image

@m4rch3n1ng m4rch3n1ng force-pushed the tree-escape-backslash branch from e4a2b27 to 31809b8 Compare November 18, 2025 01:41
@m4rch3n1ng m4rch3n1ng changed the title fix: also escape "\\" when pretty printing anonymous nodes fix: also escape '\' when pretty printing anonymous nodes Nov 18, 2025
@m4rch3n1ng m4rch3n1ng force-pushed the tree-escape-backslash branch from 31809b8 to 6f5c53f Compare November 18, 2025 01:42
@m4rch3n1ng m4rch3n1ng changed the title fix: also escape '\' when pretty printing anonymous nodes fix: also escape \ when pretty printing anonymous nodes Nov 18, 2025
@m4rch3n1ng m4rch3n1ng force-pushed the tree-escape-backslash branch from 6f5c53f to 376785a Compare November 18, 2025 01:46
@m4rch3n1ng m4rch3n1ng force-pushed the tree-escape-backslash branch from 376785a to b7dec44 Compare November 18, 2025 01:46
@the-mikedavis the-mikedavis merged commit 6721894 into helix-editor:master Nov 18, 2025
7 checks passed
@m4rch3n1ng m4rch3n1ng deleted the tree-escape-backslash branch November 18, 2025 01:53
Eucladia pushed a commit to Eucladia/helix that referenced this pull request Jan 20, 2026
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