Skip to content

unicode escapes#146

Closed
mbostock wants to merge 1 commit intomainfrom
mbostock/ascii
Closed

unicode escapes#146
mbostock wants to merge 1 commit intomainfrom
mbostock/ascii

Conversation

@mbostock
Copy link
Member

@mbostock mbostock commented Nov 18, 2025 •
edited
Loading

Ref. d3/d3#4043
@Fil
Copy link
Member

Fil commented Nov 18, 2025

Interestingly when I test this, the minification reinstates the unescaped higher-plane utf-8 characters:

grep -o "[^ -~]" dist/d3-format.min.js µ −
@mbostock
Copy link
Member Author

mbostock commented Nov 19, 2025

Oh, that’s fascinating. So we do have to set ascii_only: true whenever we use Terser? I guess it seems surprising that Terser would choose the “wrong” default… shouldn’t we just let Terser do whatever it thinks is best?
@mbostock mbostock closed this Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@mbostock @Fil