Bugfix: swap="outerHTML" on <div> with style attribute leaves htmx-swapping class behind#3341

Merged

1cg merged 11 commits into

bigskysoftware:devfrom

matthews314:swap-outer-html-bug

Jul 14, 2025

Commits on Jun 18, 2025

Added regression test for swap=outerHTML unexpected behavior, checked it failes, implemented initial fix in htmx.js that makes (all) test(s) run and pass.
Matteo Smaila
committed
Renamed variable in my regression test to be more clear.
Matteo Smaila
committed
I noticed I wasn't using the copies of the attributes I introduced.Tests were passing and I know why, though. This means I miss one more regression test for the bug in cloneAttributes.
Matteo Smaila
committed
Added one more regression test for the fix in cloneAttributes.
Matteo Smaila
committed
Made preservation of htmx- prefixed classes more robust in cloneAttributes after I noted they could as well be removed by mergeTo.setAttribute in the second forEach loop.
Matteo Smaila
committed
Started as a typo-fix, ended up renaming regression tests to be more explicit.
Matteo Smaila
committed
Started as a typo-fix, ended up renaming regression tests to be more explicit.
Matteo Smaila
committed
Removed space that I accidentally added before.
Matteo Smaila
committed
Merge branch 'dev' into swap-outer-html-bug
Matteo Smaila
committed

Commits on Jun 25, 2025

Applied changes as requested by MichaelWest22.
Matteo Smaila
committed

Commits on Jul 14, 2025

Merge branch 'dev' into swap-outer-html-bug
1cg
authored