nft preview card

Community feedback

• 

  Luka Glonti•3,440

  @lack21Posted over 1 year ago

  Great work 👍, but I have a recommendation!

  • Replace height: 100vh to min-height: 100vh in the div, the difference is that, when you set height: 100vh to something, that means it won't be bigger than that, it might cause some problems in the future, so better approach is to set min-height: 100vh, like this in case content is overflowing container will adjust to it!

  1

  ---

  

  Schismond•160

  @SchismondPosted over 1 year ago

  @lack21 thanks for the advice, I'm aware about the min-height attribute I left it at height: 100vh knowing that I won't fall into overflowing problem , but prevention is better than cure. thanks for your advice

  1