Yasmin Silva
@yasminsilva1
All comments
- Yasmin Silva• 60
@yasminsilva1
Posted
Hi @wideirp! Congratulations on your work! Your page is responsive, you are using semantic HTML, and it looks just like the original page! Your code is well structured and is 100% reusable!
1 @MarioCarlaibe
Submitted
What are you most proud of, and what would you do differently next time?
eu tive dificuldade na hora de estilizar os botões, pq havia colocado tudo dentro de div's. da próxima vez irei me atentar a esse detalhe. e outra coisa que irei mudar no flex box, eh em vez de usar justify-content: space-between, eu irei usar o justify-content: flex-start. se eu tive usado desde o inicio iria perder bem menos tempo e ficaria melhor a edição.
What challenges did you encounter, and how did you overcome them?
foi o hover, mas depois de algumas alterações no html, ai deu certo.
What specific areas of your project would you like help with?
to safe
Yasmin Silva• 60@yasminsilva1
Posted
Parabéns! Página perfeita! É acessível, o layout é idêntico ao proposto, o código está bem estruturado, parece ser fácil de reutilizar e refatorar!
0- EriveltonArruda• 60
@EriveltonArruda
Submitted
Yasmin Silva• 60@yasminsilva1
Posted
Parabéns pelo seu desafio! Há alguns ajustes que podem ser feitos para melhorar seu projeto. Infelizmente não foram utilizados elementos do HTML semântico. É interessante colocar o texto alternativo (alt="") nas imagens para melhorar a acessibilidade. O layout não funciona em vários tipos de tela. Há um scrollY na página. O tamanho do card parece correto, porém está posicionado errado. Para corrigir esse posicionamento você pode envolver todo o seu conteúdo em um container e definir ele com as seguintes propriedades: height: 100%; width: 100%; display: flex; flex-direction: column; justify-content: center; align-items: center;
Marked as helpful
0 - a-yemisi• 130
@a-yemisi
Submitted
What are you most proud of, and what would you do differently next time?
Learning text and border shadow
What challenges did you encounter, and how did you overcome them?
Trying to center the qr code component on the webpage
What specific areas of your project would you like help with?
None for now
Yasmin Silva• 60@yasminsilva1
Posted
Great job! Congratulations! One suggestion would be to use media queries to more precisely define the layout on different screen sizes.
0
