Skip to content

Fix stray NBSP indentation in src/htmx.js - #3943

Merged
MichaelWest22 merged 1 commit into
bigskysoftware:four-devfrom
marciomazza:fix-nbsp-indentation
Aug 2, 2026
Merged

Fix stray NBSP indentation in src/htmx.js#3943
MichaelWest22 merged 1 commit into
bigskysoftware:four-devfrom
marciomazza:fix-nbsp-indentation

Conversation

@marciomazza

Copy link
Copy Markdown
Contributor

Description

Line 1180 was indented with non-breaking-space characters (U+00A0) instead of regular
spaces — almost certainly a copy-paste typo from #3924. Harmless to JS engines (NBSP is
valid whitespace), but if both the embedding page and this script lack a declared charset,
Chromium decodes the script as Windows-1252, mangling those bytes into a stray "Â" and
throwing a SyntaxError before window.htmx is ever defined. Reproduced live to confirm.

Swapped the NBSPs for normal spaces.

Testing

Confirmed no NBSP left in the file. Reproduced the failure in real Chromium (page + script
both charset-less) before the fix, confirmed it's gone after. Full suite: 1615 passed / 2
flaky, unrelated failures (reran those two alone, both pass).

Checklist

  • I have read the contribution guidelines
  • I have targeted this PR against the correct branch (master for website changes, dev for
    source changes)
  • This is either a bugfix, a documentation update, or a new feature that has been explicitly
    approved via an issue
  • I ran the test suite locally (npm run test) and verified that it succeeded

@MichaelWest22
MichaelWest22 merged commit ac877ee into bigskysoftware:four-dev Aug 2, 2026
3 checks passed
@MichaelWest22

Copy link
Copy Markdown
Collaborator

Yeah thanks i'm not sure how this happened. seems something went wrong with that recent edit of mine somehow!

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