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

Template Part: Improve how the tag name attribute is handled #62785

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

gziolo
Copy link
Member

@gziolo gziolo commented Jun 24, 2024

What?

Fixes #62784.

Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Why?

Ensuring the HTML output makes sense.

How?

Run additional checks on the value of the tag name.

@gziolo gziolo added [Type] Bug An existing feature does not function as intended [Block] Template Part Affects the Template Parts Block labels Jun 24, 2024
@gziolo gziolo requested a review from aaronjorbin June 24, 2024 11:21
@gziolo gziolo requested a review from ajitbohra as a code owner June 24, 2024 11:21
Copy link

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@gziolo gziolo merged commit abe0925 into trunk Jun 24, 2024
62 of 65 checks passed
@gziolo gziolo deleted the update/template-part-tag-name branch June 24, 2024 11:30
@github-actions github-actions bot added this to the Gutenberg 18.7 milestone Jun 24, 2024
gziolo added a commit that referenced this pull request Jun 24, 2024
Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
gziolo added a commit that referenced this pull request Jun 24, 2024
Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
gziolo added a commit that referenced this pull request Jun 24, 2024
Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
gziolo added a commit that referenced this pull request Jun 24, 2024
Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
gziolo added a commit that referenced this pull request Jun 24, 2024
Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
gziolo added a commit that referenced this pull request Jun 24, 2024
Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
gziolo added a commit that referenced this pull request Jun 24, 2024
Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
gziolo added a commit that referenced this pull request Jun 24, 2024
Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
gziolo added a commit that referenced this pull request Jun 24, 2024
Only valid tags should be allowed rather than characters that can't be a part of HTML tags.

Co-authored-by: gziolo <gziolo@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Template Part Affects the Template Parts Block [Type] Bug An existing feature does not function as intended
2 participants