Skip to content

fix: afterHeadMerge event behavior#186

Open
teodord25 wants to merge 3 commits into
bigskysoftware:mainfrom
teodord25:fix/after-head-merge
Open

fix: afterHeadMerge event behavior#186
teodord25 wants to merge 3 commits into
bigskysoftware:mainfrom
teodord25:fix/after-head-merge

Conversation

@teodord25

Copy link
Copy Markdown

Description

This PR does the following:

  • fixes broken paths and references in the manual tests
  • fixes the afterHeadMerge event's evt.detail.added being empty

Htmx version:
2.0.8 from the cdn

Testing

I went through the existing manual tests multiple times after fixing them, and confirmed the event was working as expected using logging

Checklist

  • I have read the contribution guidelines
  • I ran the test suite locally (npm run test) and verified that it succeeded

- updated htmx import to use cdn
- updated head-support.js reference
- fixed test href to point to correct file

I suspect these issues were introduced when the
extensions were separated from the main repo
@netlify

netlify Bot commented Dec 13, 2025

Copy link
Copy Markdown

👷 Deploy Preview for htmx-extensions processing.

Name Link
🔨 Latest commit e1fba32
🔍 Latest deploy log https://app.netlify.com/projects/htmx-extensions/deploys/693d9e47901959000841c1fc

@netlify

netlify Bot commented Dec 13, 2025

Copy link
Copy Markdown

Deploy Preview for htmx-extensions canceled.

Name Link
🔨 Latest commit e1fba32
🔍 Latest deploy log https://app.netlify.com/projects/htmx-extensions/deploys/693d9e47901959000841c1fc

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.

1 participant