Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Recipe Page Main

@xMattRx

Desktop design screenshot for the Recipe page coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@R3ygoski

Posted

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

@xMattRx

Posted

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

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord