Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

NFT preview card component

Tiffā€¢ 480

@tifflee7784

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


all feedback are welcome :)

Community feedback

Adrianoā€¢ 34,090

@AdrianoEscarabote

Posted

Hi Tiff, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

To improve the responsiveness of the project, we can do this:

.container {
    max-width: 330px;
}

Avoid using px, If your web content font sizes are set to absolute units, such as pixels, the user will not be able to re-size the text or control the font size based on their needs. if you want to keep using px for development and then format the whole code to rem, you can use this vscode extension: [px to rem](https://marketplace.visualstudio.com/items?itemName=sainoba.px-to -rem)

The rest is great!

I hope it helps... šŸ‘

Marked as helpful

0

Tiffā€¢ 480

@tifflee7784

Posted

@AdrianoEscarabote Hi Adrino, Iā€™m good.

Thanks for your feedback. I did not notice this issue because I only focus on specific size of screen. You really find the point. Thank you very much :)

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord