Pure CSS and JS

Community feedback

• 

  Luis Colina•1,043

  @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

  1. 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

  2. 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

  3. 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
• 

  Hafizan Adli•1,190

  @hafizanadliPosted about 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