responsive landing page using flexbox, html, css and sass

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted about 2 years ago

  Hola @ElenaGonzalez2000 👋, ¡Buen trabajo al completar este desafío y bienvenida a la comunidad! 🎉

  Es un gran comienzo y una muy buena solucion. Tengo unos pocos consejos que podrian ayudarle a mejorar un poco su solucion.

  • Intente mejorar los nombre de los "botones1" y “botones2”. Le sugiero que aprenda la convención de nomenclatura BEM (inglés) para los nombres de clase CSS porque aumenta la legibilidad del código front-end y proporciona una estructura modular, además que encaja perfecto con SASS.
  • No debería tener todos los estilos en un media query '@media only screen and (min-width: 320px)", aunque es raro encontrarse un dispositivo menor a esas dimensiones, es mejor tener todo directamente y si ocupe las demas "media query", si su solución se comporta raro en tales dimensiones, no debería preocuparse mucho.
  • En lugar de usar píxeles en el tamaño de fuente, use unidades de medida relativas como rem o em. El tamaño de fuente en unidades de longitud absoluta (px) no permite a los usuarios con visión limitada cambiar el tamaño del texto en algunos navegadores. Referencia (inglés).

  Espero que esos consejos le ayuden.

  ¡Buen Trabajo!

  Marked as helpful

  2

  ---

  

  Elena Gonzalez•70

  @ElenaGonzalez2000Posted about 2 years ago

  @MelvinAguilar Hola! 😊 Muchas gracias por tu comentario, tome muy en cuenta tus consejos y los estoy poniendo en práctica (perdón por tardar en contestar es que recién me di cuenta de que se podía 😅). ¡Saludos!

  1
• 

  Adriano•38,040

  @AdrianoEscarabotePosted about 2 years ago

  Hi Elena Gonzalez, how are you?

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  To prevent the background image from breaking at higher resolutions, we can prevent this in two different ways:

  1. Add a background-repeat: repeat-x;, the image will repeat on the horizontal axis, preventing it from breaking.

  2. Add a background-size: 100% 50vmin;, the 50vmin will set its height as the page target, and 100% will make it stretch on the horizontal axis.

  Feel free to choose one of the two!

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  1

  ---

  

  Elena Gonzalez•70

  @ElenaGonzalez2000Posted about 2 years ago

  @AdrianoEscarabote Thank you very much for your comment! I will use it from now on 😄

  0