Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #1015: Specific mixture of quotes and escaped quotes (e.g. in a json string in an html attribute) breaks the html #1016

Merged
merged 6 commits into from
Nov 18, 2024

Conversation

oliverhaas
Copy link
Contributor

Pull Request Check List

Resolves: #1015

  • Added tests for changed code.
  • Updated documentation for changed code.

No updated docs due to it just being a bug.

I'm not happy that I had to move multiple things, but I didn't see a cleaner way to fix this.

Also, please squash, since I wasn't really working cleanly in my commits :).

Copy link

netlify bot commented Nov 10, 2024

Deploy Preview for djlint canceled.

Name Link
🔨 Latest commit f0bf54f
🔍 Latest deploy log https://app.netlify.com/sites/djlint/deploys/673b3470a2cb5a000847dfa4

@monosans monosans enabled auto-merge (squash) November 18, 2024 12:35
@monosans
Copy link
Member

Thanks!

@monosans monosans merged commit 0ba2a8c into djlint:master Nov 18, 2024
8 checks passed
monosans added a commit that referenced this pull request Nov 29, 2024
…. in a json string in an html attribute) breaks the html (#1016)"

This reverts commit 0ba2a8c.

Fixes #1048
@monosans
Copy link
Member

Hey, @oliverhaas. Had to revert this due to #1048. Could you please help with a solution?

@oliverhaas
Copy link
Contributor Author

Hey, @oliverhaas. Had to revert this due to #1048. Could you please help with a solution?

@monosans Oh damn. Yeah, I'll take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants