Skip to content
Submitted over 3 years ago

HTML and CSS .... Documentation too!

jaucau•80
@jaucau1
A solution to the NFT preview card component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Practice is the only way to learn more frontend development.

Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • P
    hcm•505
    @Hcmwebs

    Hi Jared,

    Great Job! Your solution looks impressive.

    With regards to some pointers, of course, if this is ok with you,

    1. try to landmark the project. Use the HTML5 header, main, footer .... to clear the accessibility issue. Check out this explanation:

    https://dequeuniversity.com/rules/axe/4.3/landmark-one-main?application=axeAPI

    1. I would use the h1- h6 hierarchy for the heading, always in acceding order. Instead of starting with an h2, I would start with an h1, styling it as per the guidelines of the project.
    2. The heading and the author's name should have a focus state on hover as per the design.

    I hope you find these of assistance.

    If you require any assistance with the above pointer, please let me do not hesitate to contact me.

    Regards,

    Hcmwebs

    Marked as helpful
  • Shashree Samuel•8,860
    @shashreesamuel

    Hey good job completing this challenge. Keep up the good work

    Your solution looks great however I think that the lower half of the card content needs some margin from the bottom using margin-bottom

    I hope this helps

    Cheers Happy coding 👍

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