Responsive page using display flex

Community feedback

• 

  Fatlind Shehu•2,230

  @fatlindshehuPosted over 2 years ago

  Hi @Nigraum,

  Nice work with the component, I would suggest some improvements:

  • Nice way of centering the component using flex
  • The eye icon color is white, on your component is not white
  • Try avoiding the use of px, moving to rem or em would be a better choice for responsive design.

  In case you need a reference, here's my solution to this challenge: SOLUTION

  Keep up the good work!!!

  Marked as helpful

  1
• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  Fala Yan, beleza? Parabéns por concluir o desafio do NFT Card.

  Posso deixar algumas sugestões em relação ao CSS?

  1.A sua solução ficou bem fiel ao design original, eu só mudaria uma coisa, trocaria de width: 275px para max-width: 275px; pra deixar o card flexível quando ele estiver em uma tela que muito estreita, se você deixar somente com width o valor da largura fica fixo e o container não fica flexível.

  .container {max-width: 275px;}

  2. Considere usar REM ao invés de PX porque daí seu fica melhor em mais tipos de tela, pixel funciona mas usando rem você aumenta o range de compatibilidade a adaptação à varias telas.

  3. Sempre olhe o painel de report, ali tem umas dicas bem boas de boas práticas e acessibilidade.

  Mandou bem, continue codando!

  Marked as helpful

  1

  ---

  

  Yan Negrão•60

  @NigraumPosted over 2 years ago

  @correlucas Obrigado pelo review, irei procurar usar mais REM, meu grande problema é que eu tenho esse hábito de usar px, estou tentando mudar isso. E mais uma vez obrigado mesmo pelas sugestões.

  0