Skip to content
Submitted almost 2 years ago

NFT Card component HTML CSS use flexbox

Muhammad Fathur Rohman•20
@moofath
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

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

Community feedback

  • Sameer Singh•560
    @SameerJS6Posted almost 2 years ago

    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!!!

    0
  • Atif Iqbal•3,200
    @atif-devPosted almost 2 years ago

    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.

    0
    Muhammad Fathur Rohman•20
    @moofathPosted almost 2 years ago

    @atif-dev thanks for your suggestion, it's a new knowledge for me

    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