Recipe Page Main

Community feedback

• 

  Bernardo Poggioni•5,280

  @R3ygoskiPosted 5 months ago

  Olá Matthew, parabéns pela conclusão do seu projeto, ele ficou muito bem feito e se assemelha muito ao design proposto, meus parabéns.

  Mas gostaria de dar algumas dicas quanto ao seu projeto. A primeira dica é relacionada ao uso do React, usar o React não é errado, mas notei que seu projeto poderia ter sido feito pelo vanilla. Por mais que o projeto não tenha a necessidade de criar uma parte lógica, seria interessante dividir ele em componentes, que dessa forma o uso do React seria justificado.

  A segunda dica é relacionado ao Sass, bom, ele em suma não está errado, mas tem abordagens melhores que poderiam manter ele mais conciso, como por exemplo, separar trechos entre arquivos parciais de Sass. E também sobre a nomenclatura que foi utilizada no seu JSX para as classes, notei que você fez algo semelhante a metodologia BEM, o que é ótimo e ajuda muito na organização, mas ele está de uma forma que faz com que fique até confuso, pois são muitos "recipe__", talvez o mais interessante seria fazer como tópicos, ao invés de usar um "recipe__instructions" utilizar um "instructions" poderia deixar mais limpo, mas claro, isso é mais uma questão de opinião.

  Outra dica é sobre o Typescript, novamente, não é errado utilizar, mas ele acabou não sendo utilizado no seu projeto, isso porque não foi necessário trabalhar com essa parte de lógica, então ele acaba sendo redundante.

  E agora uma dica mais relacionada a acessibilidade, na sua estrutura JSX você utilizou muitas tags não semânticas, isso não vai causar problemas na estrutura, mas isso afeta negativamente o SEO da página e a acessibilidade, então alterar algumas dessas <div> para <article> ou <section>, pode ser mais interessante.

  E novamente, parabéns pela conclusão do seu projeto, ele ficou bem feito, de verdade, caso tenha ficado alguma duvida quanto ao que eu disse, por favor, pergunte abaixo que tentarei ajudar da melhor forma possível.

  Marked as helpful

  0

  ---

  

  Matthew Nascimento•240

  @xMattRxPosted 5 months ago

  @R3ygoski Obrigado pelos conselhos, fiz toda a estruturação e organização necessária, agradeço novamente!

  1