github-actions · GitHub

nicolas-grekas added a commit to symfony/symfony that referenced this pull request

@nicolas-grekas

…#47715 (mpdude)
This PR was merged into the 6.4 branch.
Discussion
----------
[Form] Use self-closing `<input />` syntax again, reverting #47715
| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #53649
| License       | MIT
This PR reverts #47715 and puts the trailing slash back into `<input .../>` markup in the form themes.
The reasons are outlined in detail in #53649. Basically, the trailing slash in void elements like `<input>` is not required in HTML 5, yet it is perfectly valid and compliant with specs.
Writing markup that way is required to parse and process it with an XML parser, like `libxml2` underlying all of the PHP DOM, XML etc. extensions.
HTML 5 capable parser support has been added to PHP in php/php-src#12111 and will be available with PHP 8.4. So, we might want to remove the trailing slashes once Symfony requires at least PHP 8.4.
Commits
-------
 8c92200  Revert #47715

Read the original on github.com ↗