Skip to content

Conversation

bctiemann
Copy link
Contributor

@bctiemann bctiemann commented Oct 8, 2025

Closes: #17571

Removes experimental HTMX navigation.

Note: this leaves in place the {% formaction %} template tag as a hook for later customization, rather than removing all usages of it from templates.

Also does not remove existing non-experimental HTMX usages in forms, notifications, etc.

@pheus
Copy link
Contributor

pheus commented Oct 8, 2025

Drive‑by suggestion (not a reviewer/maintainer): In #20460 I added the docs for user preferences. Since this PR removes HTMX navigation, could we also remove it from the docs?

Source lines for reference:

### HTMX navigation (experimental)
Enables partial‑page navigation for supported views. Disable this preference if unexpected behavior is observed.

Happy to open a small docs PR if that helps. Thanks!

@bctiemann
Copy link
Contributor Author

bctiemann commented Oct 8, 2025

Nice catch, thanks! I've added that to this PR.

Copy link
Member

@jeremystretch jeremystretch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm on the fence about retaining the formaction tag. While I see potential future benefit in keeping it in place, I worry that it'll only introduce confusion in the near term.

Approving this as-is; your call whether to keep it.

@bctiemann
Copy link
Contributor Author

I'm going to go ahead and keep it as-is. We can remove it as tech debt later, meanwhile I think the comment is clear enough about the intent that a future dev ought not to be too confused.

@bctiemann bctiemann merged commit a91af99 into feature Oct 9, 2025
6 checks passed
@jeremystretch jeremystretch deleted the 17571-remove-htmx-navigation branch October 10, 2025 15:11
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.

3 participants