Skip to content

[5.x] Removed a comment from the js code output of the StaticCacher#14233

Merged
jasonvarga merged 1 commit intostatamic:5.xfrom
micahhenshaw:5.x
Mar 12, 2026
Merged

[5.x] Removed a comment from the js code output of the StaticCacher#14233
jasonvarga merged 1 commit intostatamic:5.xfrom
micahhenshaw:5.x

Conversation

@micahhenshaw
Copy link
Copy Markdown
Contributor

This is a small change, just removing the comment from the JS code output. This is a problem when the site code is minified down to one line, as the comment ends up causing everything after it to be commented out also.

eg:
Screenshot_20260312_110345

@micahhenshaw micahhenshaw changed the title Removed a comment from the js code output of the StaticCacher [5.x] Removed a comment from the js code output of the StaticCacher Mar 12, 2026
@micahhenshaw
Copy link
Copy Markdown
Contributor Author

If the comment is essential, I could also change it to: /* Recreate map in case the DOM changed. */ which would be fine as it has a comment finish tag

@jasonvarga
Copy link
Copy Markdown
Member

Weird that a minifier would have this issue, but it's no big deal to remove it.

@jasonvarga jasonvarga merged commit e2482c1 into statamic:5.x Mar 12, 2026
27 of 28 checks passed
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