Responsive NFT preview card component

Community feedback

• 

  Patrick Cuentas Mariano•260

  @PatrickCuentasPosted over 2 years ago

  I think it's better to use REM units that are based on the font size of the root element.

  Mainly for accessibility issues since users have different default sizes other than 16px for different reasons.

  And when using REM units, the size is based on that default size of each user, which offers a better experience.

  I am not an expert, but I hope you can use my suggestion for a broader search on the subject.

  Greetings ✨

  Marked as helpful

  2
• Account deleted

  Hi There, Great Job On This One.

  i notcied the element is not centered in the page properly so, you can do it with giving the body

  display:flex, justifiy-content:center, align-items:center, min-height:100vh

  Marked as helpful

  1

  ---

  

  Kevin Mason•280

  @mrfinesse47Posted over 2 years ago

  @Old1337 hi which element is it? also what would setting height have to do with centering? oh you mean vertically.. thanks. For sure i noticed this but kind of thought it was redundant as this is only a component card and not an actual website. But looking at it, it does look awkward in the comparison being so high up, so I will set it to center vertically. thanks for the comment.

  0