Skip to content

docs: add module import instructions for idiomorph extension - #3350

Merged
1cg merged 8 commits into
bigskysoftware:devfrom
jackielii:fix-idiomorph-docs
Jul 15, 2025
Merged

docs: add module import instructions for idiomorph extension#3350
1cg merged 8 commits into
bigskysoftware:devfrom
jackielii:fix-idiomorph-docs

Conversation

@jackielii

Copy link
Copy Markdown
Contributor

Summary

  • Added module import instructions to the idiomorph extension documentation
  • Resolves confusion about the correct way to import idiomorph when using modules

Test plan

  • Verified the import instruction matches the official idiomorph repository documentation
  • Documentation renders correctly

Fixes #3349

🤖 Generated with Claude Code

Fixes bigskysoftware#3349

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@MichaelWest22

Copy link
Copy Markdown
Collaborator

It looks like your change is against a very old version of dev branch from a year ago. This is a documentation update ideally should be done to the latest master branch instead

@jackielii

Copy link
Copy Markdown
Contributor Author

Hi @MichaelWest22 thanks for the review. Yes I realised that when I did the PR. The live docs at https://htmx.org/extensions/idiomorph/ is indeed very old. When I checkout dev, I found most instructions are up to date, that's why this PR is very small. But I still think it's worth adding this line because the right import for htmx is just idiomorph/htmx: it includes the idiomorph code as well as htmx extension part.

@jackielii

Copy link
Copy Markdown
Contributor Author

OK, looking at it again. I realised the import "ideomorph" shouldn't be there, probably someone else updated it. I updated it with the right import path. Also ran formatting on it.

Comment thread www/content/extensions/idiomorph.md
@jackielii
jackielii force-pushed the fix-idiomorph-docs branch from 9a5b5b9 to 1968c20 Compare July 12, 2025 12:14
@jackielii

Copy link
Copy Markdown
Contributor Author

sorry about the many commits, I cleaned up all the places where it mentions the js without htmx.

@MichaelWest22 MichaelWest22 added website Non-documentation website issues and removed needs rebase/retarget labels Jul 12, 2025
@1cg
1cg merged commit 93cafa5 into bigskysoftware:dev Jul 15, 2025
1 check passed
@1cg

1cg commented Jul 15, 2025

Copy link
Copy Markdown
Contributor

thank you @jackielii

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

website Non-documentation website issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants