Skip to content
Submitted 8 months ago

nft-preview-component-main

George Kandelakiā€¢610
@GeorgeKandelaki
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 Give me FeedBacks and tell me about my mistakes. Also Tell me what can be written better in the code, So I can get better at coding and improve my skills. Thanks!

Community feedback

  • Daniel šŸ›øā€¢44,250
    @danielmrz-devPosted 8 months ago

    Hello @GeorgeKandelaki!

    Your solution looks excelent!

    I have just one suggestion:

    šŸ“Œ To improve semantic clarity, opt for <h1> over <h3> for your main title.

    It's more than just text size ā€” it's about structuring your content effectively:

    • <h1> to <h6> are used to define HTML headings, with <h1> being the most significant.
    • Stick to one <h1> per page for the main title, and maintain the titles hierarchy with <h1>, <h2> <h3>, and so on.

    While these adjustments might not alter the visual appearance much, they significantly enhance semantic clarity, SEO optimization, and accessibility.

    Hope this suggestion proves helpful! Keep up the great work!

    Marked as helpful
    1
    George Kandelakiā€¢610
    @GeorgeKandelakiPosted 8 months ago

    @danielmrz-dev Thanks

    1

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