Preview NFT Card responsive CSS/HTML

Community feedback

• 

  Ayomide Gidigbi•100

  @DraysongzPosted over 2 years ago

  Hello sara, just went through your live site, you didn't add any active state to your design as seen in the design we were given but your design creation looks good enough. I also like the fact that you made it your own by inputting your name in the creation aspect. Your code is also well structured and i love your naming conventions, you're definitely gonna make a good front end developer but try to take notes of few things that you'll given by the UI designer.

  Marked as helpful

  0

  ---

  

  Sara Xavier•20

  @Sara-XavierPosted over 2 years ago

  @Draysongz Olá! Realmente, só coloquei os estados ativos nos textos e acabei deixando de lado o estado da imagem. Foi o primeiro projeto CSS que fiz realmente sozinha, sem a ajuda de um tutor, fiquei bem feliz com o resultado. Eu agradeço muito o seu comentário, vou cuidar um pouco melhor da endentação e anotações do código. Mais uma vez: obrigada pelo feedback!

  0
• 

  Juca Perez•270

  @jucajulian0907Posted over 2 years ago

  Olá, Sarah! Parabéns à solução do seu desafio ficou muito bom!

  O que você poderia fazer para centralizar os ícones e dar ao container que contém esses elementos estas propriedades:

  display: flex; justify-content: space-between; align-items: center;

  Voce pode testar elas, vao ficar bem centralizadas, boa sorte!

  Marked as helpful

  0

  ---

  

  Sara Xavier•20

  @Sara-XavierPosted over 2 years ago

  @jucajulian0907 Olá! Vou testar sim, muito obrigada! <3

  0
• 

  Sara Xavier•20

  @Sara-XavierPosted over 2 years ago

  Alguém consegue me explicar como a visualização do meu projeto está diferente da visualização do Site de Visualização? O código é o mesmo!

  0

  ---

  

  Ayomide Gidigbi•100

  @DraysongzPosted over 2 years ago

  @Sara-Xavier no usaste el tamaño del escritorio al diseñar el tamaño de escritorio es 1440px

  Marked as helpful

  0