@correlucas
Posted
Oi Camila, tudo bem?
Tenho algumas dicas pra você:
1.Vi que você usou em e rem e isso é mto bom, usando essas medidas relativas ao invés de px se você tiver que escalar o tamanho dos elementos depois ou o usuário der zoom na tela, todos itens vão crescer proporcionalmente, eu só te recomendo usar uma medida só, nesse caso vc pode usar sempre REM. Outra dica é usar um plugin do VSCODE chamado PX TO REM, que converte automaticamente todos elementos escritos em PX para REM, basta selecionar e apertar alt + z. Aqui o link: https://marketplace.visualstudio.com/items?itemName=sainoba.px-to-rem
2.O seu componente já está com o design pronto, um bom exercício pra melhorar seu código, seria criar a estrutura mais limpa o possível e reduzir seu código ao máximo. Por exemplo, você pode segurar todo conteúdo em um bloco só usando (<main> ou <div>) nesse caso vamos usar <main> por que é uma tag semantica. Se você notar, nem precisa adicionar nenhum div além da principal pra segurar o conteúdo (img, h1 e p). A estrutura ideal e mais limpa possível pra esse desafio é essa aqui:
<body>
<main>
<img src="./images/image-qr-code.png" alt="QR Code Frontend Mentor" >
<h1>Improve your front-end skills by building projects</h1>
<p>Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p> </main>
</body>
Para reduzir o CSS, você pode excluir todas as classes e selecionar todos elementos no CSS com o seletor direto para cada um deles, nesse caso (img, div, main, h1 e p).
Faltou também colocar o alt text pra imagem ser reconhecida por leitores de tela.
2.Você definiu o tamanho do card com max-width e isso é bom, pois deixa o container responsive, mas por conta de vc ter colocado o width dentro do html, a imagem ficou com aquele tamanho fixo, o ideal seria vc usar max-width: 100% pra imagem ter o tamanho de 100% do elemento pai e não crescer mais do que isso e diminuir juntamente com o resto de acordo com a tela. Lembre-se de usar sempre max-width quando vc quiser que um elemento tenha um tamanho limite e possa diminuir, ou seja, responsivo, use width somente quando vc quiser que ele tenha tamanho fixo.
img {
max-width: 100%;
}
3.Para separar os elementos, vc pode usar sempre padding-bottom ou margin-bottom para separar verticalmente os elementos, desse jeito vc adiciona esse propriedade de cima pra baixo e quando precisar mudar algo basta olhar o elemento de cima. Para criar um espaçamento de padding uniforme com a imagem vc pode definir um padding pro container de 16px por exemplo pra criar uma borda interna entre todos elementos, desse jeito vc so precisa se preocupar com os paddings verticais.
Aqui minha solução caso você queira usar como referência: https://www.frontendmentor.io/solutions/qr-code-component-vanilla-cs-js-darklight-mode-nS2aOYYsJR
Espero ter ajudado e continue no foco, vc já começou muito bem, a maioria das pessoas não acertam o design e o seu ficou perfeito, só fazer alguns ajustes e reduzir a estrutura do html deixando o codigo mais limpo e vai ficar perfeito. Parabéns ✌
Marked as helpful
