Skip to content
Submitted over 2 years ago

NFT Card component HTML CSS use flexbox

Muhammad Fathur Rohman•20
@moofath
A solution to the NFT preview card component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Please help me to review my coding, I will accept all suggestions from all of you. Thank You

Code

Please log in to post a comment

Log in

Community feedback

  • Sameer Singh•560
    @SameerJS6

    Congratulations on completing this little fun challenge and you have done a great job.

    • But, you have to focus on the Accessibility report & HTML validation report, as it helps in using the clean and standard practices for writing code.

    Good Luck!!!

  • P
    Atif Iqbal•3,320
    @atif-dev

    Hi Muhammad Fathur Rohman, congrats🎉 on completing the challenge. Better take care about following points.

    • Always check Frontendmentor Report Generator issues after submitting the project for removing errors and warnings. To overcome warning: Page should contain a level-one heading. Create level one heading (<h1></h1>) inside main and hide it visually. Although it will be hidden on UI but will be readable for screen readers technology. To avoid accessibility issue "All page content should be contained by landmarks" use code as :
    <body>
    <main>
        ---your code here----
    </main>
  <footer>
  </footer>
</body>

    (why main matters? Read here)

    Hope you will find this Feedback Helpful.

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