Remove all references to "self-closing" void elements in core

Added another related issue 📌 Remove trailing slashes on void elements Closed: duplicate .

Briefly discussed this with @longwave and @laurii during Drupal Dev Days 2023 and decided to mark 📌 Remove trailing slashes on void elements Closed: duplicate as duplicate. I'm copying the work that I did there over to this issue.

It's probably still worth waiting for 🐛 Upgrade filter system to HTML5 Fixed to get in first.

I have attached patch for 11.x, please review

The last submitted patch, 106: 1388926-11.x-106.patch, failed testing. View results →

Fixed the one remaining test failure.

#105 onwards has lost all the work done in #86 and before. There are still plenty of instances of <br /> and other void tags in core, perhaps we need to break this out into multiple issues.

Hope this issue will help to progress with related 🐛 Newline character in HtmlTag causes unwanted whitespace Needs work

#108 is already a very large patch to review. So think expanding on it would be even more difficult. So think breaking off into smaller tickets make sense. Maybe by tag?

Moving to NW for the follow ups to be created and issue summary to be updated with the new plan (if agreed upon). #108 is a good start so maybe after the follow ups we still merge this?

@smustgrave

#108 is already a very large patch to review. So think expanding on it would be even more difficult. So think breaking off into smaller tickets make sense. Maybe by tag?

Why not open a MR? it is easy to review without open new issues.

Breaking this into smaller issues is not about whether it is a patch or an MR. It is about the scope and patch size. There is a limit to what a reviewer can effectively review and once past that limit, mistakes are made. And we can help avoid those mistakes by splitting this into smaller pieces. There is more information about scope and patch size → in the wiki.

I was looking at the patch in #86 and I do agree with smustgrave that doing this by tag is worth pursuing. However, this must be balanced if there are only a few instances of some tags that need changing. It could be sensible to combine some tags together while still keeping the change size small.

I think that thanks to 🐛 Upgrade filter system to HTML5 Fixed , this now became actually feasible to do!

In that context would great to tackle 🐛 Newline character in HtmlTag causes unwanted whitespace Needs work

Patch for 10.3.x versions