Skip to content
Submitted almost 2 years ago

CSS respond

Abdokhalil11•560
@Abdokhalil11
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...NFT preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

I Wish It can like You

Community feedback

  • Sandro•1,150
    @sandro21-glitchPosted almost 2 years 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
    0
  • Coder Suresh•870
    @coderSureshPosted almost 2 years 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
    0

Please log in to post a comment

Log in with GitHub

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