Skip to content

refactor(templates): update djlint to 1.45.1 and fix issues - #39231

Open
silverwind wants to merge 2 commits into
go-gitea:mainfrom
silverwind:djlint-1.45
Open

refactor(templates): update djlint to 1.45.1 and fix issues#39231
silverwind wants to merge 2 commits into
go-gitea:mainfrom
silverwind:djlint-1.45

Conversation

@silverwind

@silverwind silverwind commented Sep 3, 2026

Copy link
Copy Markdown
Member
  • Add explicit type to every <button>, submit where it submits its form, button elsewhere (H043)
  • Replace <br> spacing with paragraphs and a margin (H036)
  • Double-quote attributes and trim template tag whitespace (H008, T032)
  • Add the doctype to the footer's dummy block (H007)

To ease review, branch has 2 comments, one for buttons, one for the rest.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 3, 2026
@github-actions github-actions Bot added the type/refactoring Existing code has been cleaned up. There should be no new functionality. label Sep 3, 2026
@silverwind
silverwind force-pushed the djlint-1.45 branch 2 times, most recently from 026dec3 to a9e517b Compare September 3, 2026 23:35
@silverwind

Copy link
Copy Markdown
Member Author

djlint/djLint#2452 was merged and just released, I will update to that and remove the workaround.

A button without a type is a submit button, so a click or Enter in a
form did whatever the markup happened to imply. Buttons that submit
their form say submit, everything else says button. Also covers
djlint's new H043 rule.

Assisted-by: Claude Code:claude-fable-5-1
H045 is ignored because the only iframe is the devtest mail preview.
H046 is ignored because the sign-in form orders its tab stops on
purpose.

Assisted-by: Claude Code:claude-fable-5-1
@silverwind silverwind changed the title refactor(templates): update djlint to 1.45.0 and resolve its new findings refactor(templates): update djlint to 1.45.1 and resolve its new findings Sep 4, 2026
@silverwind silverwind changed the title refactor(templates): update djlint to 1.45.1 and resolve its new findings refactor(templates): update djlint to 1.45.1 and fix issues Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/refactoring Existing code has been cleaned up. There should be no new functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants