Remove trailing slashes on void elements

This patch simply changes the closing tag for void elements from /> to > on line 91 of core/lib/Drupal/Core/Render/Element/HtmlTag.php.

Some code refactoring in core/lib/Drupal/Core/Render/Element/HtmlTag.php and fixing automated test fails.

The idea makes sense but my concern with making such a change could cause issues for existing sites. Maybe both scenarios should be supported with a warning for the old way?

Going to see what the committers think.

The last submitted patch, 6: 3346188-6-10.1.x.patch, failed testing. View results →

Seems random

I think this is probably OK, it would likely only fail if a site specifies xhtml output which I doubt any are doing these days. However putting in front of the Front end framework managers for a look.

There seems to be a related issue: 📌 Remove all references to "self-closing" void elements in core Needs work (with a broader scope probably).

The last submitted patch, 6: 3346188-6-10.1.x.patch, failed testing. View results →

I was about to reroll #6, but after discussing briefly with @laurii and @longwave, I've decided to mark this issue as duplicate of 📌 Remove all references to "self-closing" void elements in core Needs work . The work done in that issue probably covers all void elements everywhere, including the code covered here.

Let's continue the work there!

This happen in Drupal 10 also.

Uploaded patch in #6 with 10.2 compatibility for projects that rely on this patch, until related Issue is fixed, or a valid solution is found.

Wrong patch filename, fixing it.

Thanks for the patch, Juanjol. However, this issue was marked Closed (duplicate).

I suggest we continue to discuss/work in [1388926], starting with the discussion about how we can improve the issue by dividing the work needed into seperate issues with a smaller scope.

Uploaded patch in #19 with 10.2.4 compatibility.