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

HTML and CSS .... Documentation too!

Jaber Aljoodiā€¢ 80

@jaaljoodi

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


Practice is the only way to learn more frontend development.

Community feedback

P
hcmā€¢ 505

@Hcmwebs

Posted

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

1
Shashree Samuelā€¢ 9,260

@shashreesamuel

Posted

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 šŸ‘

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