Design comparison
Community feedback
- @denieldenPosted about 2 years ago
Hi Valmir, congratulations on completing the challenge, great job! 😁
Some little tips for optimizing your code:
- use
article
tag instead of a simplediv
to the container card for improve the Accessibility - add descriptive text in the
alt
attribute of the images - You can add the effect
:hover
creating adiv
that appears on hover. I used tailwind but you can still see and understand which css properties you can use to do the same. Look here -> my solution - remove all
margin
frommain
tag - remove all
padding
from body - use flexbox to the body to center the card. Read here -> best flex guide
- after, add
min-height: 100vh
to body because Flexbox aligns child items to the size of the parent container - instead of using
px
use relative units of measurement likerem
-> read here
Hope this help! Happy coding 😉
Marked as helpful0@valmirsantanepePosted about 2 years ago@denielden Excelentes dicas.... Eu comecei estudar posicionamento de elementos usando flexbox e tbm unidades de medidas relativas, mas por esta naquela fase inicial ainda estou sentindo um pouco de dificuldade na hora de aplicar os conceitos na praticas. Muito obrigado pela disponibilidade em ajudar. Sempre que puder comente meus códigos. É muito bom ter pessoas mais experientes ajudando no processo de aprendizado.
1@denieldenPosted about 2 years ago@valmirsantanepe You are welcome and keep it up 😁
0 - use
- @correlucasPosted about 2 years ago
Oi Valmir,
Sua página do desafio não tá online por causa de um erro no Github Pages. Acabei de ver que você está tendo alguns problemas para configurar seu preview site Github Pages. Nos meus primeiros dias, tentei usar o Github Pages também, mas era muito chato de configurar.
Se você quiser continuar usando o Github Pages, aqui é um guia que ensina configurar passo a passo:
https://docs.github.com/en/pages/quickstart
.Meu conselho se você quiser configurar seu site rápido e sem stress, meu conselho é você usar o
vercel.com
ounetlify.com
que são plataformas realmente mais fáceis de usar para live sites e são totalmente user-friendly, em questão de 5min tá tudo pronto. Tudo o que você precisa fazer é conectar a conta do Github, importar o repositório e fazer o deploy. O melhor deles éVercel
e é o que uso para meus desafios.Corrigindo isso, você precisa atualizar a solução com o novo link e poderemos ver a solução e te ajudar.
Marked as helpful0@correlucasPosted about 2 years ago@valmirsantanepe Só atualizar o link da solução e mandar bala irmao! Jaja eu olho
Marked as helpful1@valmirsantanepePosted about 2 years ago@correlucas
Oi Lucas, acabei de ver aqui que realmente está ocorrendo esse problema, porem no github pages a pagina esta disponível.
Segue o link: https://valmirsantanepe.github.io/nft-preview-card-component-main/
Agradeço mais um vez pelo feedback e não deixe de comentar meus códigos.... :D Forte abraço!
0
Please log in to post a comment
Log in with GitHubJoin 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