The challenge was completed with the following skills: HTML5, CSS3, JS

Community feedback

• 

  P. Ricardo•2,370

  @pRicard0

  Posted over 1 year ago

  Projeto ficou muito bom, parabéns, pretendo me inspirar um pouco e fazer umas features diferente. Mas aqui vai algumas coisinhas que eu notei, não são muito relevantes mas talvez você queira saber.

  Diquinha HTML.

  • Para um HTML mais semântico, recomendo que ao invés de você utilizar a tag <p> contendo o preço "$250.00" recomendo que você utilize a tag semântica <s>. Essa tag significa strikethrough, tachado. Ela vem por padrão com esse risco no meio. Ela é para texto que não é mais relevante, não é mais preciso e etc... o que se encaixa nesse caso.
  • Outra tag que dá para utilizar no lugar do <p> é a tag <em> para o texto de desconto "50%". Essa tag significa emphasis, um texto que você queira dar ênfase. E o texto de desconto está recebendo estilos adicionais, dando uma ideia de estar enfatizado( você também poderia aplicar para o paragrafo com o texto $125.00 )
  • Como você usou um svg dentro de um paragrafo, é recomendado que você adicione dois atributos para que impedir que ele possa ser acessível por dispositivos auxiliares de pessoas com deficiência, esses seriam... aria-hidden="true" e focusable="false"
  • No svg do carrinho de compras na parte de cima, você fez certo, adicionou o atributo alt para um svg que não é apenas ilustrativo, mas recomendo tirar o hífen já que provavelmente o leitor de tela iria ler com hífen. Você pode ser mais descritivo também.
  • Você pode englobar toda a tag <ul> do menu com uma <nav>

  Talvez eu esteja sendo meio chato demais já que teu proposito foi só melhorar pratica com Redux, mas é bom que tu faça algo acessível e semântico já no automatico e da forma mais natural possível.

  01

  Marked as helpful