Skip to content
OverviewSolutions
4
Comments
4
Yasmin Silva
@yasminsilva1

All comments

  • Yasmin Silva•60
    @yasminsilva1Posted 3 months ago

    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
  • MARIO CARLAIBE DE ANDRADE•110
    @MarioCarlaibeSubmitted 4 months ago
    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

    flex box e tag

    1
    Yasmin Silva•60
    @yasminsilva1Posted 4 months ago

    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
  • Yasmin Silva•60
    @yasminsilva1Posted 4 months ago

    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•110
    @a-yemisiSubmitted 4 months ago
    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

    Frontend Mentor - QR code component solution

    2
    Yasmin Silva•60
    @yasminsilva1Posted 4 months ago

    Great job! Congratulations! One suggestion would be to use media queries to more precisely define the layout on different screen sizes.

    0