Submitted about 1 year ago

NFT Card component HTML CSS use flexbox

Muhammad Fathur Rohman• 20

@moofath

Preview site View code

This is a solution for...

NFT preview card component

HTML
CSS

1newbie

View challenge

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

@SameerJS6

Posted about 1 year 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!!!

Atif Iqbal• 3,220

@atif-dev

Posted about 1 year 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.

Muhammad Fathur Rohman• 20

@moofath

Posted about 1 year ago

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

Please log in to post a comment

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