order-summary-component-main using HTML/CSS Flexbox

Community feedback

• 

  Alan Santos•50

  @alan-ssantosPosted about 3 years ago

  Oi Thainá, Tudo bem? 😉

  Adorei o trabalho feito nesse desafio, mas tenho algumas sugestões que talvez possam ajudar a deixar sua solução ainda melhor.

  • Toda página deve ter ao menos um titulo h1 e os níveis não devem ser pulados, como usar h1 e em seguida h3. Sendo assim você poderia mudar o "Order Summary" para h1 e o "Annual Plan" para h2 ou p. Neste link você pode ver mais sobre.

  • O icone de música deve ter o atributo alt vazio [alt=""] e deve conter aria-hidden="true", pois ele é apenas um elemento decorativo, não sendo relevante para leitores de tela. Mais sobre aria-hidden aqui e sobre alt-text aqui.

  • Considere alterar o background de no-repeat para repeat-x, pois em telas maiores a imagem é exibida apenas no centro da página.

  • Por último eu recomendo não misturar português e inglês na nomenclatura das classes, para não gerar confusão de quem for revisar o código.

  Espero que essas dicas possam ser úteis, continue fazendo os desafios e se precisar de algo é só chamar. 😁✌

  Marked as helpful

  1

  ---

  

  Thainá Lopes•110

  @thainadlopesPosted about 3 years ago

  @alan-ssantos As dicas foram muito úteis, vou fazer as demais alterações! Muito obrigada! : )

  0
• 

  Alif•40

  @alifjsPosted about 3 years ago

  Hey I did this project too a little while ago!

  1