QR Code Component Main using HTML and CSS
Solution retrospective
How would you optimize this code? What would you change about this code? What best practices would you have implemented?
¿Cómo optimizarías este código? ¿Qué cambiarías de este código? ¿Qué mejores prácticas hubieras implementado?
Please log in to post a comment
Log in with GitHubCommunity feedback
- @samaelwebdev
Hola,
Algo que yo en lo personal cambiaría sería lo siguiente:
actualiza el “README” file y agrega tu propio contenido, facilitará la navegación de tus proyectos para las personas que visiten tus repositorios.
Crea un folder con nombre “CSS” y dentro de ese folder crea tu stylesheet, esto como mejor práctica ya que en un futuro probablemente uses más de un stylesheet.
Procura usar unidades como REM en lugar de pixels y esto ayudará a la hora de hacer tu proyecto “responsive”
También procura utilizar Max-width, Max-height en lugar de width y height y esto ayudará también en cuanto a hacer tu proyecto “responsive”
Excellente diseñó, suerte y feliz coding.
Marked as helpful - @vanzasetia
Hi, Ayrton Bolaños! 👋
Nice work on getting the HTML correct! It's rare to see people coming up with a good HTML structure for this challenge. 👍
The QR code needs an alternative text because it is the main content of the page. Make sure you describe the purpose of the QR code. Don't include the word "image" on it since the
<img>element has already the image semantic meaning.I hope this helps. Keep up the good work! Happy coding! 😄
Marked as helpful
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord