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

Challenge 1 - Frontend Mentor

@jaojogadez

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello, I'm 16 years old and I'm new to the area, I was introduced to Front-end in high school and I found it very interesting and fun and I'm happy to have found this site where I can practice HTML and CSS with challenges, thank you very much. Regarding my first project, I had more difficulty than I thought I would have, but I managed to solve it in the end, I was satisfied with the result.

Community feedback

@irenanrodrigues

Posted

Fala @jaojogadez, tudo bem? Parabéns por completar o desafio. Alem das dicas do Ricardo, tenho mais algumas sugestões para melhorar seu código, se você aceitar.

Na imagem você acabou colocando 2 width, recomendo remover um e definir o tamanho para width: 100%, assim a imagem fica responsiva e acompanha o tamanho do seu card.

Na sua classe .content, recomendo você remover o flexbox, ele apenas está centralizando a imagem. Você pode colocar um padding: 20px;.

Sempre que estiver desenvolvendo, deixe aberto o DevTools, do seu navegador, para ver o comportamento das propriedades css.

Marked as helpful

0

@jaojogadez

Posted

@irenanrodrigues muito obrigado pelas sugestões e dicas bro!

0
P. Ricardo 2,370

@pRicard0

Posted

Dicas HTML

  • Não é recomendado utilizar hífen para a descrição do atributo alt. Pense no atributo alt como a maneira que uma pessoa cega iria entender aquela imagem.
  • Você deveria utilizar h1 ao invés de h3, não pula a hierarquia dos títulos. Enquanto não houver h1, não há h2 e assim por diante.
  • Você deveria utilizar a tag <p> ao invés de um h4 para o paragrafo.

Dicas CSS

  • O conteúdo não está muito bem centralizado, isso porque você definiu uma altura para o body, o que é errado. Altura fixa vai fazer com que o conteúdo só esteja centralizado naquela tela mas se diminuir de tamanho ou aumentar o tamanho da tela vai ficar muito mal. Para centralizar, você tem que tirar tanto a width e a height que você colocou. Você só precisa botar os seguintes estilos: margin: 0; e min-height: 100vh;

Marked as helpful

0

@jaojogadez

Posted

@pRicard0 Obrigado pelas dicas, apliquei elas e facilitou muito, muito obrigado.

0

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