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

Avoid trailing slash on void elements. #460

Merged

Conversation

jaimeiniesta
Copy link
Contributor

Although void elements are allowed to have a trailing slash, the W3C Validator will issue a warning in that case, as it's now recommended to avoid trailing slash on void elements:

Captura de pantalla 2023-04-25 a las 12 49 03

More information here:

https://github.com/validator/validator/wiki/Markup-%C2%BB-Void-elements

Although void elements are allowed to have a trailing slash, the W3C Validator
will issue a warning in that case:

https://github.com/validator/validator/wiki/Markup-%C2%BB-Void-elements
@RobertDober
Copy link
Collaborator

¡Muchìsimas gracias por ese PR!

...
I'll prbably just ridicule myself in castellano and maybe others need to read this too, therefore...

Firstlly thanx this surely has great value and will be released soon
Secondly I am happy with many of the format changes you made and can tolerate some although I dislike them personally
And last but not least: I really dislike some changes of text in the tests so would you be ok to revert some of them, I will make the remarks in a CR as soon as I have time

@jaimeiniesta
Copy link
Contributor Author

Hola @RobertDober, muchísimas gracias a tí :)

Thanks for reviewing this, of course, feel free to revert some of the changes if needed.

Format changes were not intended, and they were applied automatically by my editor. But as those come from the official mix format I thought it would be OK to leave them.

Let me know if you want me to change anything but yes, feel free to revert changes or improve the PR.

@RobertDober
Copy link
Collaborator

RobertDober commented Apr 25, 2023 via email

@RobertDober
Copy link
Collaborator

Hola again

unfortunately the CI is making problems, so I will just merge and change some things back in the tests, if the tests are ok I will release if I have any problems I'll come back to you.

Supongo que los tests funccionen en(?) tu maquina local?

@RobertDober RobertDober merged commit ccaa204 into pragdave:master Apr 25, 2023
@jaimeiniesta
Copy link
Contributor Author

Hey yes, the tests pass in my machine :D

@jaimeiniesta
Copy link
Contributor Author

CI is waiting for a runner 🏃‍♀️

@RobertDober
Copy link
Collaborator

Excellente

I had authorized the runner \o/ what happened

aùn gracias y voy sicuramente hacer un release este weekend prolongado(sp?)

@RobertDober
Copy link
Collaborator

Ah finally I understand your changes in the tests too! 👍 Getting old I guess 😢

@RobertDober
Copy link
Collaborator

Published

@jaimeiniesta
Copy link
Contributor Author

Great! Thanks! 🥸

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants