Design comparison
Solution retrospective
any feedback is appreciated.
Thanks
Community feedback
- @Comet466Posted about 3 years ago
gran trabajo Renato tu interfas se ve genial y responde bien a distintos tamaños de pantalla, voy a señalar un par de cosas a tomar en cuenta para mejorar aun mas tu solucion
-
los elementos de los navegadores tienen un outline predeterminado que para muy pocos casos seria el mas adecuado para los colores que se estan utilizando en el layout, esto afecta directamente a tus usuarios que usen el teclado para moverse por tu sitio, puedes reiniciar estos elementos con la pseudo clase :focus e interactuando con la regla 'outline:' para modificarlo por algo mas acorde y vistoso
-
no es considerado una practica recomendada el uso de la propiedad innerHTML ya que conlleva muchos riesgos de seguridad y hasta conflictos con los demas elementos de la pagina, para estos casos se recomienda ampliamente usar textContent en su lugar
-
tienes muchos errores de accesibilidad Renato, esto en una practica para frontend no tiene ninguna repercusion pero ten en cuenta que el motor de busqueda de google (SEO) evalua esos errores y disminuye la exposicion del sitio, asi que seria una buena practica debuggear el layout.
pero vamos que te quedo muy bonito de verdad, un abrazo y happy coding (<= solo suena bien en ingles)
0 -
- @hafizanadliPosted over 3 years ago
What a great work!
Your site looks good, very close to design and responsive in any screen
The part you can improve is to use semantic html.
Keep up the good work!
0
Please log in to post a comment
Log in with GitHubJoin 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