Submitted over 2 years ago

NFT-PREVIEW-CARD solution

Mikhael Opeyemi-Olatunji• 150

@mikhael-oo

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

NFT preview card component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

Please provide general feedback on what I could do better in this project. I just started with web development and it would do me a ton of good to know how I could better grow in my journey of learning. Thank you.

Community feedback

Fernando FeN• 425

@Fernando0654

Posted over 2 years ago

👋 Hey! I saw you used max-width property with %. This isn't a good idea because it will get smaller on smaller screen sizes. The best way of doing this is:

max-width: 600px and width: 95% for example. This way, it's not gonna grow more than 600px on bigger screens and it'll be 95% on any kind of mobile device

Good luck, browww.

Marked as helpful

Mikhael Opeyemi-Olatunji• 150

@mikhael-oo

Posted over 2 years ago

@Fernando0654 ah thanks. I also thought of using media queries but your suggestion might be the better option.

Anosha Ahmed• 9,300

@anoshaahmed

Posted over 2 years ago

To get rid of the accessibility/HTML issues shown in your Report, wrap everything in your body in <main> OR use semantic tags OR give role="" to the direct children of your <body> ... Click here to read more

Good job! :)

Marked as helpful

Mikhael Opeyemi-Olatunji• 150

@mikhael-oo

Posted over 2 years ago

@anoshaahmed Thank you!!

Please log in to post a comment

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