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

Always render script element as plain HTML #1239

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

Archmonger
Copy link
Contributor

Description

Checklist

Please update this checklist as you complete each item:

  • Tests have been developed for bug fixes or new functionality.
  • The changelog has been updated, if necessary.
  • Documentation has been updated, if necessary.
  • GitHub Issues closed by this PR have been linked.

By submitting this pull request I agree that all contributions comply with this project's open source license(s).

@Archmonger Archmonger force-pushed the fix-script-rendering branch from 5dee01a to 23c894f Compare October 31, 2024 02:16
@Archmonger
Copy link
Contributor Author

Current issue with this PR is that script elements are not re-running when the content changes.

This may be a good thing and a healthier direction for us to go in, but it needs to be investigated whether this behavior is due to a larger bug with the new design.

@Archmonger Archmonger added the release-major Warrents a major release label Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-major Warrents a major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant