Submitted over 1 year ago

CSS respond

Abdokhalil11• 540

@Abdokhalil11

Preview site View code

This is a solution for...

NFT preview card component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

I Wish It can like You

Community feedback

Sandro• 1,170

@sandro21-glitch

Posted over 1 year ago

Hi Abdokhalil

Here are a few suggestions for improvement

Use semantic HTML elements: Consider using semantic HTML elements such as header and footer to improve the accessibility of your code.

Make class names descriptive: Consider using class names that accurately describe the purpose of the element, such as card-header instead of photo.

Add alt text: Make sure to add descriptive alt text to the images in the code.

Add hover effects for the card: To make the card more interactive, you can add hover effects to change the background color or add an animation effect on hover.

Happy Coding

Marked as helpful

Coder Suresh• 870

@coderSuresh

Posted over 1 year ago

Hey Abdokhalil11, nice work. I am noticing some issues here. You can solve some of those by wrapping everything inside <body> tag inside of the <main> tag.

You can validate your HTML code through following URL:

https://validator.w3.org/

Go to the link, click on validate by direct input and paste your code. You can upload your HTML file also. It'll show you issues in your code (if any) along with possible solution.

Marked as helpful

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