Community feedback

• 

  Candela Bramucci•680

  @bramuccciPosted almost 3 years ago

  Hola Santiago! Ese error en tu desafío anterior de debe a que no usaste ningún h1 en tu html. Como es una tarjeta aislada creo que la decisión de un h2 es correcta. Solo recuerda que cuando hagas una página web entera tenés que incluir un y solo un h1. Sobre este desafío, tengo varias sugerencias:

  • Tu breakpoint es de 1100 px. Es innecesariamente grande y deja a las pantallas de 1024px y 768px (laptop y tablet) de lado. Te recomiendo usar 600px y leer este artículo
  • En el body pusiste font-weight: 400px;
  • Tu card está envuelta solamente por el body. Una tarjeta es un único elemento. Deberías usar un elemento (article, por ejemplo) para toda la tarjeta y usar divs para dividirla. Mi estructura sería article>div*3>h3+p+button
  • Usar flexbox en el body te va a dar problemas en cuanto quieras agregar otros elementos

  Espero que estos tips te ayuden :) Keep coding!

  1

  ---

  

  Santiago Carranza•20

  @santiagocarranz-aPosted almost 3 years ago

  @bramuccci encantando de toda la información que recibí ,muchas gracias Candela. Lo iré corrigiendo!

  1
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! Good Job Santiago Carranza

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body like it should be your container

  -> For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc

  -> But use header tag only once in main heading element.

  -> Always use h1 first and then h2, h3 and so on

  Keep up the good work!

  1

  ---

  

  Santiago Carranza•20

  @santiagocarranz-aPosted almost 3 years ago

  @Crazimonk muchisimas gracias Naveen

  1